summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDaniel Jasper <djasper@google.com>2014-09-19 08:01:25 +0000
committerDaniel Jasper <djasper@google.com>2014-09-19 08:01:25 +0000
commita41aa536dc389f95763adfca3cafdf4adbe353b2 (patch)
tree06709d5925d6e674f02a3cc71d50d108158cc12f
parent752c9cb12ff6d6e50042d8178bf42c882ed580c1 (diff)
downloadbcm5719-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.cpp7
-rw-r--r--clang/unittests/Format/FormatTest.cpp5
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) {
OpenPOWER on IntegriCloud