[PATCH] D42684: clang-format: Allow optimizer to break template declaration.

Francois Ferrand via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 28 09:24:32 PST 2018


Typz added a comment.

Indeed, seems to apply to classes as well. Maybe I was mislead by my testing, where I did not get the case (possibly because we use `ConstructorInitializerAllOnOneLineOrOnePerLine=true`, so the continuation indenter only sees "short" class declarations unless breaking the template is required because the name is too long).

So just to be clear, are you saying the whole approach is not the right one, or simply that the "names" of each modes are not?
For the name, maybe something like may be better:

- `Never`
- `MultiLineDeclaration`, or maybe even `MultiLine`
- `Always`


Repository:
  rC Clang

https://reviews.llvm.org/D42684





More information about the cfe-commits mailing list