diff options
Diffstat (limited to 'unittests/Format/FormatTest.cpp')
| -rw-r--r-- | unittests/Format/FormatTest.cpp | 21 |
1 files changed, 8 insertions, 13 deletions
diff --git a/unittests/Format/FormatTest.cpp b/unittests/Format/FormatTest.cpp index cc68bcb947..98e035a598 100644 --- a/unittests/Format/FormatTest.cpp +++ b/unittests/Format/FormatTest.cpp @@ -857,29 +857,24 @@ TEST_F(FormatTest, EndOfFileEndsPPDirective) { } TEST_F(FormatTest, IndentsPPDirectiveInReducedSpace) { - // If the macro fits in one line, we still do not get the full - // line, as only the next line decides whether we need an escaped newline and - // thus use the last column. - verifyFormat("#define A(B)", getLLVMStyleWithColumns(13)); - - verifyFormat("#define A( \\\n B)", getLLVMStyleWithColumns(12)); - verifyFormat("#define AA(\\\n B)", getLLVMStyleWithColumns(12)); + verifyFormat("#define A(BB)", getLLVMStyleWithColumns(13)); + verifyFormat("#define A( \\\n BB)", getLLVMStyleWithColumns(12)); verifyFormat("#define A( \\\n A, B)", getLLVMStyleWithColumns(12)); + // FIXME: We never break before the macro name. + verifyFormat("#define AA(\\\n B)", getLLVMStyleWithColumns(12)); verifyFormat("#define A A\n#define A A"); verifyFormat("#define A(X) A\n#define A A"); - verifyFormat("#define Something Other", getLLVMStyleWithColumns(24)); - verifyFormat("#define Something \\\n" - " Other", - getLLVMStyleWithColumns(23)); + verifyFormat("#define Something Other", getLLVMStyleWithColumns(23)); + verifyFormat("#define Something \\\n Other", getLLVMStyleWithColumns(22)); } TEST_F(FormatTest, HandlePreprocessorDirectiveContext) { EXPECT_EQ("// some comment\n" "#include \"a.h\"\n" - "#define A(A,\\\n" - " B)\n" + "#define A( \\\n" + " A, B)\n" "#include \"b.h\"\n" "// some comment\n", format(" // some comment\n" |