[PATCH] clang-format: Support @synchronized.

Daniel Jasper djasper at google.com
Tue Jun 23 04:23:57 PDT 2015


================
Comment at: docs/ClangFormatStyleOptions.rst:420
@@ -419,1 +419,3 @@
 
+**ObjCSpaceAfterSynchronized** (``bool``)
+  Add a space after ``@synchronized`` in Objective-C, i.e. use
----------------
Actually, I don't think we should have this option at all (and possibly not the one above either, but I can try to deprecate that later). Would it work for you to bind this to what is selected for SpaceBeforeParens? Specifically, can we handle this like control statements? If not, can we extend SpacesBeforeParens to have more enum values?

================
Comment at: lib/Format/UnwrappedLineParser.cpp:662
@@ +661,3 @@
+      nextToken();
+      if (FormatTok->Tok.is(tok::l_paren))
+        parseParens();
----------------
Remove "Tok." (and below).

http://reviews.llvm.org/D10371

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list