HazardyKnusperkeks added inline comments.
================ Comment at: clang/lib/Format/UnwrappedLineParser.cpp:2303 + assert(IfRightBrace->MatchingParen == IfLeftBrace); + IfLeftBrace->MatchingParen = nullptr; + IfRightBrace->MatchingParen = nullptr; ---------------- owenpan wrote: > HazardyKnusperkeks wrote: > > Why null that? > `MatchingParen` is used to flag braces that may be optional. If it turns out > that they should not be removed, we must reset it. Otherwise, the test case > on line 23314 in FormatTest.cpp below would fail. Is `MatchingParen` for (if) braces null before this patch? Because I would expect that always to be set, if there is a matching brace and maybe base patches on that assumption. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116316/new/ https://reviews.llvm.org/D116316 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits