[PATCH] D14484: [clang-format] Formatting constructor initializer lists by putting them always on different lines

Nikolai Hlubek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 21 07:53:08 PDT 2019


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





More information about the llvm-commits mailing list