[PATCH] Added an option to allow short function bodies be placed on a single line.

Manuel Klimek klimek at google.com
Wed Nov 20 08:34:24 PST 2013


  lg


================
Comment at: lib/Format/Format.cpp:439
@@ +438,3 @@
+                 : 0;
+    }
+    if (TheLine->InPPDirective &&
----------------
Alexander Kornienko wrote:
> Manuel Klimek wrote:
> > Which test breaks that needed the change from an if () {} else if () {} ... return 0; to doing the check within each if?
> I didn't intend to change the behavior, only to make the control flow more clear. And added code is laid out the same way.
Ok... (I think we might get a test to break, but then again, it looks simple enough)


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

BRANCH
  svn

ARCANIST PROJECT
  clang



More information about the cfe-commits mailing list