[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