[PATCH] D105099: [clang-format] Add an option to put one constructor initializer per line

[MyDeveloperDay via Phabricator via cfe-commits]

MyDeveloperDay added a comment.

> is a smell that tells us to use an enum.

I totally agree

> Personally I find it harder to finder multiple options that concern the same aspect of the code style rather than a single option.

To be honest, I think we know the options some of the best (apart of course from @djasper and @klimek), and most of the time I'm totally confused, as to which options do which, it ends up being trail and error. This way bugs creep in.

Yes I feel that way too this should have been

ConstructorInitializer:  Compact

or

ConstructorInitializer:  OnePerLine

It would make


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105099/new/

https://reviews.llvm.org/D105099