diff options
| author | Daniel Jasper <djasper@google.com> | 2013-01-23 15:55:19 +0000 |
|---|---|---|
| committer | Daniel Jasper <djasper@google.com> | 2013-01-23 15:55:19 +0000 |
| commit | 13cb7c2c46bca6f52fd217091ef2922a3eca46f5 (patch) | |
| tree | f5d76666feb2e5987bb99e70c3141baf3817e4f9 /unittests/Format/FormatTest.cpp | |
| parent | 7ccbc2156bfe473f080b585130d70e51c4b62bb0 (diff) | |
Don't try to do a hanging ident after assignments.
Before:
bool aaaa = aaaaaaaaaaa(
aaaaaaaaaaaaaaaaa);
After:
bool aaaa = aaaaaaaaaaa(
aaaaaaaaaaaaaaaaa);
The other indentation was a nice attempt but doesn't work in many cases.
Not sure what the right long term solution is as the "After: " is still
not nice. We either need to figure out what to do in the cases where it
"doesn't work" or come up with a third solution, e.g. falling back to:
bool aaaa =
aaaaaaaaaaa(
aaaaaaaaaaaaaaaaa);
which should always work and nicely highlight the structure.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@173268 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'unittests/Format/FormatTest.cpp')
| -rw-r--r-- | unittests/Format/FormatTest.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/unittests/Format/FormatTest.cpp b/unittests/Format/FormatTest.cpp index 708b7d8086..493f2ad7b2 100644 --- a/unittests/Format/FormatTest.cpp +++ b/unittests/Format/FormatTest.cpp @@ -1040,9 +1040,8 @@ TEST_F(FormatTest, BreaksAfterAssignments) { " TTI.getMemoryOpCost(I->getOpcode(), VectorTy, SI->getAlignment(),\n" " SI->getPointerAddressSpaceee());\n"); verifyFormat( - "CharSourceRange LineRange =\n" - " CharSourceRange::getTokenRange(Line.Tokens.front().Tok.getLoc(),\n" - " Line.Tokens.back().Tok.getLoc());"); + "CharSourceRange LineRange = CharSourceRange::getTokenRange(\n" + " Line.Tokens.front().Tok.getLo(), Line.Tokens.back().Tok.getLoc());"); } TEST_F(FormatTest, AlignsAfterAssignments) { |