[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