diff options
| author | Daniel Jasper <djasper@google.com> | 2013-01-28 07:35:34 +0000 |
|---|---|---|
| committer | Daniel Jasper <djasper@google.com> | 2013-01-28 07:35:34 +0000 |
| commit | 7b5773e9c771f5e922675c241570dc6ea2853abf (patch) | |
| tree | 067a9dcf8231a8a908d3a0b3c5a282705f1c3e1b /clang/unittests/Format/FormatTest.cpp | |
| parent | f0705254d15855569dd48972d59d563b4a61cf06 (diff) | |
| download | bcm5719-llvm-7b5773e9c771f5e922675c241570dc6ea2853abf.tar.gz bcm5719-llvm-7b5773e9c771f5e922675c241570dc6ea2853abf.zip | |
Avoid confusing identations for multi-parameter functions.
Before:
aaaaaaaa(aaaaaaaaa(
aaaaaaaaaa(),
aaaaaaaaa);
After:
aaaaaaaa(aaaaaaaaa(
aaaaaaaaaa(),
aaaaaaaaa);
llvm-svn: 173673
Diffstat (limited to 'clang/unittests/Format/FormatTest.cpp')
| -rw-r--r-- | clang/unittests/Format/FormatTest.cpp | 36 |
1 files changed, 28 insertions, 8 deletions
diff --git a/clang/unittests/Format/FormatTest.cpp b/clang/unittests/Format/FormatTest.cpp index 7d274832a30..d877dc29e55 100644 --- a/clang/unittests/Format/FormatTest.cpp +++ b/clang/unittests/Format/FormatTest.cpp @@ -796,13 +796,15 @@ TEST_F(FormatTest, LayoutStatementsAroundPreprocessorDirectives) { " a;", format("int\n#define A\na;")); verifyFormat( - "functionCallTo(someOtherFunction(\n" - " withSomeParameters, whichInSequence,\n" - " areLongerThanALine(andAnotherCall,\n" + "functionCallTo(\n" + " someOtherFunction(\n" + " withSomeParameters, whichInSequence,\n" + " areLongerThanALine(andAnotherCall,\n" "#define A B\n" - " withMoreParamters,\n" - " whichStronglyInfluenceTheLayout),\n" - " andMoreParameters), trailing);", getLLVMStyleWithColumns(69)); + " withMoreParamters,\n" + " whichStronglyInfluenceTheLayout),\n" + " andMoreParameters), trailing);", + getLLVMStyleWithColumns(69)); } TEST_F(FormatTest, LayoutBlockInsideParens) { @@ -849,11 +851,29 @@ TEST_F(FormatTest, FormatsFunctionDefinition) { TEST_F(FormatTest, FormatsAwesomeMethodCall) { verifyFormat( - "SomeLongMethodName(SomeReallyLongMethod(\n" - " CallOtherReallyLongMethod(parameter, parameter, parameter)),\n" + "SomeLongMethodName(SomeReallyLongMethod(CallOtherReallyLongMethod(\n" + " parameter, parameter, parameter)),\n" " SecondLongCall(parameter));"); } +TEST_F(FormatTest, HigherIndentsForDeeperNestedParameters) { + verifyFormat( + "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa(\n" + " aaaaaaaaaaaaaaaaaaaaaaaa(\n" + " aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa),\n" + " aaaaaaaaaaaaaaaaaaaaaaaa);"); + verifyFormat( + "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa[\n" + " aaaaaaaaaaaaaaaaaaaaaaaa[\n" + " aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa],\n" + " aaaaaaaaaaaaaaaaaaaaaaaa];"); + verifyFormat( + "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa<\n" + " aaaaaaaaaaaaaaaaaaaaaaaa<\n" + " aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa>,\n" + " aaaaaaaaaaaaaaaaaaaaaaaa>;"); +} + TEST_F(FormatTest, ConstructorInitializers) { verifyFormat("Constructor() : Initializer(FitsOnTheLine) {}"); verifyFormat("Constructor() : Inttializer(FitsOnTheLine) {}", |
