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

owenpan added a comment. In D14484#4087879 , @HantaoPan wrote: > Thank you! Is there a similar flag wrt function parameter? (say a "Never" > other than "false", which can always put each function parameter on its own > line.) Nope. Repository: rL

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

HantaoPan added a comment. In D14484#4087827 , @owenpan wrote: > In D14484#4079711 , @HantaoPan wrote: > >> Hi, >> I am interesting in this feature too. You know, consistency is crucial to a >> large program... >>

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

owenpan added a comment. In D14484#4079711 , @HantaoPan wrote: > Hi, > I am interesting in this feature too. You know, consistency is crucial to a > large program... > > regards, As mentioned above, it has been added in D108752

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

HantaoPan added a comment. Herald added a project: All. Hi, I am interesting in this feature too. You know, consistency is crucial to a large program... regards, Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D14484/new/ https://reviews.llvm.org/D14484

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

owenpan added a comment. See D108752 . 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

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

nhlubek added a comment. In order to push this forward I have written a bug ticket that highlights this issue: https://bugs.llvm.org/show_bug.cgi?id=50549 The patch is also really simple with the clang-format options that are available nowadays

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

Jeroen added a comment. In D14484#2788953 , @FStefanni wrote: > Hi, > > in case of any doubt: yet it is of interest. > 6 years old means only that it has not been "fixed" in the meanwhile... > > Please consider that there are not many code formatters for

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

FStefanni added a comment. Hi, in case of any doubt: yet it is of interest. 6 years old means only that it has not been "fixed" in the meanwhile... Please consider that there are not many code formatters for C++, since it is quite a complex language (and a lot of different formatting

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

MyDeveloperDay added a comment. We can't rename options without giving some form of backwards compatibility, personally, I think we need to start again with this review if its still of interest, this one is 6 years old. Repository: rL LLVM CHANGES SINCE LAST ACTION

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

tksuoran added a comment. I would like to use exactly the same formatting (really exactly one initializer (and nother else) per line). Some comments: - Keep `ConstructorInitializerAllOnOneLineOrOnePerLine` so that the change does not break any existing behavior - Default value for

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

catskul added a comment. @FStefanni, after managing to update this patch to work with the latest code, and trying out your example it appears this patch doesn't quite cover your case. I did manage to fix it to cover your case, but I suppose at this point I should ask @MyDeveloperDay and

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

FStefanni added a comment. Hi, thank you for the suggestion, but it does **not**, at least with `BreakConstructorInitializers: AfterColon` (which is what I use). If initializers are "small enough" to fit the constructor line, all will finish on the same line. E.g. MyClass::MyClass(): a(),

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

catskul added a comment. In D14484#2246171 , @FStefanni wrote: > Hi to all, > > I am also interested to this option, since match my personal style, but more > important, in my experience, this kind of formatting is very used. > I hope it will be

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

FStefanni added a comment. Herald added a subscriber: danielkiss. Hi to all, I am also interested to this option, since match my personal style, but more important, in my experience, this kind of formatting is very used. I hope it will be implemented in a near future. Which is the current

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

nhlubek added a comment. Required change for break after colon: https://github.com/llvm-mirror/clang/pull/67 New unit test shows use case that is not covered currently. Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D14484/new/ https://reviews.llvm.org/D14484

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

nhlubek added a comment. In D14484#1689271 , @MyDeveloperDay wrote: > Looking at this I'm wondering if this Isn't at least partially handled by the > `BreakConstructorInitializersStyle` in combination with >

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

puya added a comment. > Looking at this I'm wondering if this Isn't at least partially handled by the > BreakConstructorInitializersStyle in combination with > ConstructorInitializerAllOnOneLineOrOnePerLine style? I’m fairly certain that’s only true when line is long enough to be broken. In

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

MyDeveloperDay requested changes to this revision. MyDeveloperDay added a comment. This revision now requires changes to proceed. Looking at this I'm wondering if this Isn't at least partially handled by the `BreakConstructorInitializersStyle` in combination with