[PATCH] Don't merge constructors with initializer lists to one line in WebKit style.

Daniel Jasper djasper at google.com
Sat Dec 21 06:28:54 PST 2013



================
Comment at: lib/Format/Format.cpp:522
@@ +521,3 @@
+            if (I + 2 != E)
+              I[2]->First->MustBreakBefore = true;
+            return 0;
----------------
I don't really like this. This means that when we try to merge lines and fail suddenly has side-effects. Could we reverse this (i.e. change MustBreakBefore when we do merge the lines)?


http://llvm-reviews.chandlerc.com/D2455



More information about the cfe-commits mailing list