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 @djasper, can/should I upload my diff to replace this one? I'd like to adopt this diff and get it past the finish line if possible. In D14484#2338756 <https://reviews.llvm.org/D14484#2338756>, @FStefanni wrote: > 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. > > ... 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
