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

Tue Jun 29 14:32:47 PDT 2021

owenpan added a comment.

In D105099#2847933 <https://reviews.llvm.org/D105099#2847933>, @HazardyKnusperkeks wrote:

> In D105099#2847328 <https://reviews.llvm.org/D105099#2847328>, @MyDeveloperDay wrote:
>
>> Seem similar to D90232: [clang-format] Formatting constructor initializer lists by putting them always on different lines (update to D14484) <https://reviews.llvm.org/D90232> which seems to have got stalled
>>
>> I sort of feel I prefer the design where we have an enum rather than introducing a separate option.
>
> Yeah I thought, I already saw something like that.
>
> I too think the way with the enum is a better one.

I was aware of that one. Didn't it get blocked because it failed the unit tests? Besides, it didn't handle the `AfterColon` case.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105099