[PATCH] clang-format: Support @autoreleasepool.
strager
strager.nds at gmail.com
Thu Jun 25 17:20:10 PDT 2015
================
Comment at: lib/Format/UnwrappedLineParser.cpp:660
@@ -659,1 +659,3 @@
return;
+ case tok::objc_autoreleasepool:
+ nextToken();
----------------
djasper wrote:
> (I think this is also what Nico means): Join this case and the next. I think it makes sense to format "@autoreleasepool (foo) {" like @synchronized, even if it might not be valid syntax. If you care about not having parens here, extend the corresponding if-statement.
Ah, I thought Nico was talking about the test cases, not this `switch`.
I agree.
http://reviews.llvm.org/D10372
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list