[PATCH] Support GNU style rule to put a space before opening parenthesis.

Alexander Kornienko alexfh at google.com
Mon Dec 9 11:30:46 PST 2013


  Here's a quick response to one of the comments. I'll work on others tomorrow.


================
Comment at: unittests/Format/FormatTest.cpp:2085
@@ -2084,2 +2084,3 @@
 TEST_F(FormatTest, RespectWhitespaceInMacroDefinitions) {
-  verifyFormat("#define A (1)");
+  EXPECT_EQ("#define A (x)", format("#define A (x)"));
+  EXPECT_EQ("#define A(x)", format("#define A(x)"));
----------------
Daniel Jasper wrote:
> I think having this test once is sufficient (it is also below).
If you mean the one at lines 6703-6704, it's not redundant, it tests SpaceBeforeParens: Always.


http://llvm-reviews.chandlerc.com/D2326

BRANCH
  svn

ARCANIST PROJECT
  clang



More information about the cfe-commits mailing list