[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