owenpan added inline comments.
================ Comment at: clang/lib/Format/UnwrappedLineParser.cpp:2303 + assert(IfRightBrace->MatchingParen == IfLeftBrace); + IfLeftBrace->MatchingParen = nullptr; + IfRightBrace->MatchingParen = nullptr; ---------------- HazardyKnusperkeks wrote: > 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. Yes, it's not set until it's in the token annotator. That's why I added the assertion in the annotator. 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