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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
