================
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/



_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to