diff options
| author | Daniel Jasper <djasper@google.com> | 2013-09-08 14:07:57 +0000 |
|---|---|---|
| committer | Daniel Jasper <djasper@google.com> | 2013-09-08 14:07:57 +0000 |
| commit | 9b246e09ced67f228724699e18a10034c6f62c2b (patch) | |
| tree | e4a6201bad33400b964ceac75bcfcc7127208fb1 /clang/unittests/Format | |
| parent | 77029be19a22b33f18a0c0c3c8d66f642699e161 (diff) | |
| download | bcm5719-llvm-9b246e09ced67f228724699e18a10034c6f62c2b.tar.gz bcm5719-llvm-9b246e09ced67f228724699e18a10034c6f62c2b.zip | |
clang-format: Keep empty lines and format 1-line nested blocks.
Let clang-format consistently keep up to one empty line (configured via
FormatStyle::MaxEmptyLinesToKeep) in nested blocks, e.g. lambdas. Also,
actually format single statements in nested blocks.
Before:
DEBUG({ int i; });
DEBUG({
int i;
// an empty line here would just be removed.
int j;
});
After:
DEBUG({ int i; });
DEBUG({
int i;
int j;
});
llvm-svn: 190278
Diffstat (limited to 'clang/unittests/Format')
| -rw-r--r-- | clang/unittests/Format/FormatTest.cpp | 15 |
1 files changed, 14 insertions, 1 deletions
diff --git a/clang/unittests/Format/FormatTest.cpp b/clang/unittests/Format/FormatTest.cpp index 10a6316054b..ef7d63c1ebb 100644 --- a/clang/unittests/Format/FormatTest.cpp +++ b/clang/unittests/Format/FormatTest.cpp @@ -2276,6 +2276,19 @@ TEST_F(FormatTest, LayoutNestedBlocks) { " somethingElse();\n" "});", getLLVMStyleWithColumns(29))); + EXPECT_EQ("DEBUG({ int i; });", format("DEBUG({ int i; });")); + EXPECT_EQ("DEBUG({\n" + " int i;\n" + "\n" + " // comment\n" + " int j;\n" + "});", + format("DEBUG({\n" + " int i;\n" + "\n" + " // comment\n" + " int j;\n" + "});")); } TEST_F(FormatTest, PutEmptyBlocksIntoOneLine) { @@ -6395,7 +6408,7 @@ TEST_F(FormatTest, FormatsLambdas) { verifyFormat("auto c = { [&a, &a, a] {\n" " [=, a, b, &c] { return b++; }();\n" "} }\n"); - verifyFormat("auto c = { [&a, &a, a] { [=, a, b, &c] { }(); } }\n"); + verifyFormat("auto c = { [&a, &a, a] { [=, a, b, &c] {}(); } }\n"); verifyFormat("void f() {\n" " other(x.begin(), x.end(), [&](int, int) { return 1; });\n" "}\n"); |

