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