Index: clang/lib/Format/UnwrappedLineFormatter.cpp =================================================================== --- clang/lib/Format/UnwrappedLineFormatter.cpp +++ clang/lib/Format/UnwrappedLineFormatter.cpp @@ -404,8 +404,9 @@ // If possible, merge the next line's wrapped left brace with the // current line. Otherwise, leave it on the next line, as this is a // multi-line control statement. - return (Style.ColumnLimit == 0 || - TheLine->Last->TotalLength <= Style.ColumnLimit) + return (Style.ColumnLimit == 0 || TheLine->Level * Style.IndentWidth + + TheLine->Last->TotalLength <= + Style.ColumnLimit) ? 1 : 0; } Index: clang/unittests/Format/FormatTest.cpp =================================================================== --- clang/unittests/Format/FormatTest.cpp +++ clang/unittests/Format/FormatTest.cpp @@ -3096,6 +3096,14 @@ " baz);", format("do{foo();}while(bar&&baz);", Style)); // Long lines should put opening brace on new line. + verifyFormat("void f() {\n" + " if (a1 && a2 &&\n" + " a3)\n" + " {\n" + " quux();\n" + " }\n" + "}", + "void f(){if(a1&&a2&&a3){quux();}}", Style); EXPECT_EQ("if (foo && bar &&\n" " baz)\n" "{\n" @@ -3210,7 +3218,6 @@ format("try{foo();}catch(...){baz();}", Style)); Style.BraceWrapping.AfterFunction = true; - Style.BraceWrapping.AfterControlStatement = FormatStyle::BWACS_MultiLine; Style.AllowShortFunctionsOnASingleLine = FormatStyle::SFS_All; Style.ColumnLimit = 80; verifyFormat("void shortfunction() { bar(); }", Style);