nhlubek added a comment.

In D14484#1689271 <https://reviews.llvm.org/D14484#1689271>, @MyDeveloperDay 
wrote:

> Looking at this I'm wondering if this Isn't at least partially handled by the 
> `BreakConstructorInitializersStyle`  in combination with 
> `ConstructorInitializerAllOnOneLineOrOnePerLine` style?




> [...]



> At least the unit tests appear to be covered by using those styles?

I checked with https://zed0.co.uk/clang-format-configurator/ and it isn't 
handled.

The reason is that `BreakConstructorInitializersStyle` only comes into play if 
the constructor initializers don't fit on a line. 
The unit tests work because they have

  Style.ColumnLimit = 60;




Repository:
  rL LLVM

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

https://reviews.llvm.org/D14484



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to