diff options
| author | Daniel Jasper <djasper@google.com> | 2013-01-23 16:58:21 +0000 |
|---|---|---|
| committer | Daniel Jasper <djasper@google.com> | 2013-01-23 16:58:21 +0000 |
| commit | a836b90aa735b897e919ed4bae2e2757d0aafe5e (patch) | |
| tree | 0f85e12a10cc44382ab227e50ede5b8eec105025 /clang/unittests/Format/FormatTest.cpp | |
| parent | b53f37c97882dbce1b717b40ec5bcc5630b2784d (diff) | |
| download | bcm5719-llvm-a836b90aa735b897e919ed4bae2e2757d0aafe5e.tar.gz bcm5719-llvm-a836b90aa735b897e919ed4bae2e2757d0aafe5e.zip | |
Don't try to align builder-type continuations on assignments.
Before:
int aaaa = aaaaa().aaaaa() // force break
.aaaaa();
After:
int aaaa = aaaaa().aaaaa() // force break
.aaaaa();
The other indent is just wrong and confusing.
llvm-svn: 173273
Diffstat (limited to 'clang/unittests/Format/FormatTest.cpp')
| -rw-r--r-- | clang/unittests/Format/FormatTest.cpp | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/clang/unittests/Format/FormatTest.cpp b/clang/unittests/Format/FormatTest.cpp index 493f2ad7b2e..a588dbabece 100644 --- a/clang/unittests/Format/FormatTest.cpp +++ b/clang/unittests/Format/FormatTest.cpp @@ -1006,10 +1006,10 @@ TEST_F(FormatTest, FormatsOneParameterPerLineIfNecessary) { TEST_F(FormatTest, FormatsBuilderPattern) { verifyFormat( "return llvm::StringSwitch<Reference::Kind>(name)\n" - " .StartsWith(\".eh_frame_hdr\", ORDER_EH_FRAMEHDR)\n" - " .StartsWith(\".eh_frame\", ORDER_EH_FRAME).StartsWith(\".init\", ORDER_INIT)\n" - " .StartsWith(\".fini\", ORDER_FINI).StartsWith(\".hash\", ORDER_HASH)\n" - " .Default(ORDER_TEXT);\n"); + " .StartsWith(\".eh_frame_hdr\", ORDER_EH_FRAMEHDR)\n" + " .StartsWith(\".eh_frame\", ORDER_EH_FRAME).StartsWith(\".init\", ORDER_INIT)\n" + " .StartsWith(\".fini\", ORDER_FINI).StartsWith(\".hash\", ORDER_HASH)\n" + " .Default(ORDER_TEXT);\n"); } TEST_F(FormatTest, DoesNotBreakTrailingAnnotation) { @@ -1042,6 +1042,10 @@ TEST_F(FormatTest, BreaksAfterAssignments) { verifyFormat( "CharSourceRange LineRange = CharSourceRange::getTokenRange(\n" " Line.Tokens.front().Tok.getLo(), Line.Tokens.back().Tok.getLoc());"); + + verifyFormat( + "aaaaaaaaaaaaaaaaaaaaaaaaaa aaaa = aaaaaaaaaaaaaa(0).aaaa()\n" + " .aaaaaaaaaaaa(aaaaaaaaaaaaaaaaaaa::aaaaaaaaaaaaaaaaaaaaa);"); } TEST_F(FormatTest, AlignsAfterAssignments) { |
