diff --git a/clang/lib/Format/UnwrappedLineParser.cpp b/clang/lib/Format/UnwrappedLineParser.cpp --- a/clang/lib/Format/UnwrappedLineParser.cpp +++ b/clang/lib/Format/UnwrappedLineParser.cpp @@ -578,14 +578,17 @@ // BlockKind later if we parse a braced list (where all blocks // inside are by default braced lists), or when we explicitly detect // blocks (for example while parsing lambdas). + // FIXME: Some of these do not apply to JS, e.g. "} {" can never be a + // braced list in JS. ProbablyBracedList = (Style.Language == FormatStyle::LK_JavaScript && NextTok->isOneOf(Keywords.kw_of, Keywords.kw_in, Keywords.kw_as)) || (Style.isCpp() && NextTok->is(tok::l_paren)) || NextTok->isOneOf(tok::comma, tok::period, tok::colon, - tok::r_paren, tok::r_square, tok::ellipsis) || - (NextTok->isOneOf(tok::l_brace, tok::identifier) && + tok::r_paren, tok::r_square, tok::l_brace, + tok::ellipsis) || + (NextTok->is(tok::identifier) && !PrevTok->isOneOf(tok::semi, tok::r_brace, tok::l_brace)) || (NextTok->is(tok::semi) && (!ExpectClassBody || LBraceStack.size() != 1)) || @@ -2861,7 +2864,7 @@ // class Foo implements {bar: number} { } nextToken(); if (FormatTok->is(tok::l_brace)) { - parseBracedList(); + tryToParseBracedList(); continue; } } diff --git a/clang/unittests/Format/FormatTest.cpp b/clang/unittests/Format/FormatTest.cpp --- a/clang/unittests/Format/FormatTest.cpp +++ b/clang/unittests/Format/FormatTest.cpp @@ -11867,27 +11867,6 @@ " f(v);\n" "}"); - verifyFormat("void foo() {\n" - " { // asdf\n" - " { int a; }\n" - " }\n" - " {\n" - " { int b; }\n" - " }\n" - "}"); - verifyFormat("namespace n {\n" - "void foo() {\n" - " {\n" - " {\n" - " statement();\n" - " if (false) {\n" - " }\n" - " }\n" - " }\n" - " {}\n" - "}\n" - "} // namespace n"); - // Long lists should be formatted in columns even if they are nested. verifyFormat( "vector x = function({1, 22, 333, 4444, 55555, 666666, 7777777,\n"