diff options
| author | Daniel Jasper <djasper@google.com> | 2013-07-09 11:57:27 +0000 |
|---|---|---|
| committer | Daniel Jasper <djasper@google.com> | 2013-07-09 11:57:27 +0000 |
| commit | bd05888fa0a3c600e7a75704d38f786b8fe549cf (patch) | |
| tree | db06040efb7e48eed883e1a12aab8e09d15bb260 /clang/unittests/Format/FormatTest.cpp | |
| parent | 0f12aa2b0f5c919148efa76e755af580581fdb5d (diff) | |
| download | bcm5719-llvm-bd05888fa0a3c600e7a75704d38f786b8fe549cf.tar.gz bcm5719-llvm-bd05888fa0a3c600e7a75704d38f786b8fe549cf.zip | |
Avoid confusing indentations for chained function calls.
Basically treat a function with a trailing call similar to a function
with multiple parameters.
Before:
aaaaaaaaaaaaaaaaaaaa(aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa(
aaaaaaaaaaaaaaaaaa, aaaaaaaaaaaaaaaaa))
.aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa();
After:
aaaaaaaaaaaaaaaaaaaa(aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa(
aaaaaaaaaaaaaaaaaa, aaaaaaaaaaaaaaaaa))
.aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa();
Also fix typo.
llvm-svn: 185930
Diffstat (limited to 'clang/unittests/Format/FormatTest.cpp')
| -rw-r--r-- | clang/unittests/Format/FormatTest.cpp | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/clang/unittests/Format/FormatTest.cpp b/clang/unittests/Format/FormatTest.cpp index 417c50cd2cc..dda16c52494 100644 --- a/clang/unittests/Format/FormatTest.cpp +++ b/clang/unittests/Format/FormatTest.cpp @@ -3051,6 +3051,15 @@ TEST_F(FormatTest, WrapsAtFunctionCallsIfNecessary) { " aaaaaaaaaaaaaaaaaaa,\n" " aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa);", NoBinPacking); + + // If there is a subsequent call, change to hanging indentation. + verifyFormat( + "aaaaaaaaaaaaaaaaaaaa(aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa(\n" + " aaaaaaaaaaaaaaaaaa, aaaaaaaaaaaaaaaaa))\n" + " .aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa();"); + verifyFormat( + "aaaaaaaaaaaaaaaaaaaa(aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa(\n" + " aaaaaaaaaaaaaaaaaa, aaaaaaaaaaaaaaaaa));"); } TEST_F(FormatTest, WrapsTemplateDeclarations) { @@ -3787,7 +3796,7 @@ TEST_F(FormatTest, LayoutCxx11ConstructorBraceInitializers) { verifyFormat("DoSomethingWithVector({ {} /* No data */ }, { { 1, 2 } });"); verifyFormat( "someFunction(OtherParam, BracedList{\n" - " // comment 1 (Forcing intersting break)\n" + " // comment 1 (Forcing interesting break)\n" " param1, param2,\n" " // comment 2\n" " param3, param4\n" |
