diff options
author | Daniel Jasper <djasper@google.com> | 2014-09-19 08:01:25 +0000 |
---|---|---|
committer | Daniel Jasper <djasper@google.com> | 2014-09-19 08:01:25 +0000 |
commit | a41aa536dc389f95763adfca3cafdf4adbe353b2 (patch) | |
tree | 06709d5925d6e674f02a3cc71d50d108158cc12f | |
parent | 752c9cb12ff6d6e50042d8178bf42c882ed580c1 (diff) | |
download | bcm5719-llvm-a41aa536dc389f95763adfca3cafdf4adbe353b2.tar.gz bcm5719-llvm-a41aa536dc389f95763adfca3cafdf4adbe353b2.zip |
clang-format: Undo r216377.
It has proven to not be a food idea in many case.
llvm-svn: 218107
-rw-r--r-- | clang/lib/Format/ContinuationIndenter.cpp | 7 | ||||
-rw-r--r-- | clang/unittests/Format/FormatTest.cpp | 5 |
2 files changed, 4 insertions, 8 deletions
diff --git a/clang/lib/Format/ContinuationIndenter.cpp b/clang/lib/Format/ContinuationIndenter.cpp index 0dc65ba28ef..ce6ebd84b6a 100644 --- a/clang/lib/Format/ContinuationIndenter.cpp +++ b/clang/lib/Format/ContinuationIndenter.cpp @@ -867,10 +867,9 @@ void ContinuationIndenter::moveStatePastScopeOpener(LineState &State, getColumnLimit(State)) BreakBeforeParameter = true; } - bool NoLineBreak = - State.Stack.back().NoLineBreak || - ((Current.NestingLevel != 0 || Current.Type == TT_TemplateOpener) && - State.Stack.back().ContainsUnwrappedBuilder); + bool NoLineBreak = State.Stack.back().NoLineBreak || + (Current.Type == TT_TemplateOpener && + State.Stack.back().ContainsUnwrappedBuilder); State.Stack.push_back(ParenState(NewIndent, NewIndentLevel, State.Stack.back().LastSpace, AvoidBinPacking, NoLineBreak)); diff --git a/clang/unittests/Format/FormatTest.cpp b/clang/unittests/Format/FormatTest.cpp index 7da0b534be3..93c58348a5f 100644 --- a/clang/unittests/Format/FormatTest.cpp +++ b/clang/unittests/Format/FormatTest.cpp @@ -3831,12 +3831,9 @@ TEST_F(FormatTest, FormatsBuilderPattern) { " .aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa<\n" " aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa>();"); - // Prefer not to break after empty parentheses ... + // Prefer not to break after empty parentheses. verifyFormat("FirstToken->WhitespaceRange.getBegin().getLocWithOffset(\n" " First->LastNewlineOffset);"); - // ... unless nested. - verifyFormat("f(FirstToken->WhitespaceRange.getBegin()\n" - " .getLocWithOffset(First->LastNewlineOffset));"); } TEST_F(FormatTest, BreaksAccordingToOperatorPrecedence) { |