[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