[PATCH] D43114: clang-format: fix formatting of ObjC @synchronized blocks

Ben Hamilton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 15 08:19:39 PST 2018


benhamilton accepted this revision.
benhamilton added a comment.
This revision is now accepted and ready to land.

Just a question on the test.



================
Comment at: unittests/Format/FormatTestObjC.cpp:193-198
+  verifyFormat("@synchronized(self) {\n"
+               "  f();\n"
+               "}\n"
+               "@synchronized(self) {\n"
+               "  f();\n"
+               "}\n");
----------------
Why is the block repeated twice?


================
Comment at: unittests/Format/FormatTestObjC.cpp:200-207
+  verifyFormat("@synchronized(self)\n"
+               "{\n"
+               "  f();\n"
+               "}\n"
+               "@synchronized(self)\n"
+               "{\n"
+               "  f();\n"
----------------
Same question.


Repository:
  rC Clang

https://reviews.llvm.org/D43114





More information about the cfe-commits mailing list