diff --git a/clang/lib/Format/ContinuationIndenter.cpp b/clang/lib/Format/ContinuationIndenter.cpp index 0dc65ba28efe..ce6ebd84b6aa 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 7da0b534be3f..93c58348a5f1 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) {