diff --git a/clang/lib/Format/ContinuationIndenter.cpp b/clang/lib/Format/ContinuationIndenter.cpp index 0ca297a5f95768..6b9b5aca7a364a 100644 --- a/clang/lib/Format/ContinuationIndenter.cpp +++ b/clang/lib/Format/ContinuationIndenter.cpp @@ -1437,6 +1437,7 @@ unsigned ContinuationIndenter::moveStateToNextToken(LineState &State, if (Style.PackConstructorInitializers > FormatStyle::PCIS_BinPack) { CurrentState.AvoidBinPacking = true; CurrentState.BreakBeforeParameter = + Style.ColumnLimit > 0 && Style.PackConstructorInitializers != FormatStyle::PCIS_NextLine && Style.PackConstructorInitializers != FormatStyle::PCIS_NextLineOnly; } else { diff --git a/clang/unittests/Format/FormatTest.cpp b/clang/unittests/Format/FormatTest.cpp index 1a1109f281b442..3d20163973d749 100644 --- a/clang/unittests/Format/FormatTest.cpp +++ b/clang/unittests/Format/FormatTest.cpp @@ -7758,6 +7758,13 @@ TEST_F(FormatTest, AllowAllConstructorInitializersOnNextLine) { " aaaaaaaaaaaaaaaaaaaa(a),\n" " bbbbbbbbbbbbbbbbbbbbb(b) {}", Style); + + Style = getLLVMStyleWithColumns(0); + Style.PackConstructorInitializers = FormatStyle::PCIS_CurrentLine; + verifyFormat("Foo(Bar bar, Baz baz) : bar(bar), baz(baz) {}", Style); + verifyNoChange("Foo(Bar bar, Baz baz)\n" + " : bar(bar), baz(baz) {}", + Style); } TEST_F(FormatTest, AllowAllArgumentsOnNextLine) {