owenpan added a comment.
I have second thoughts about linking block braces in `UnwrappedLineParser`.
Consider the following example:
#if A
while (a) { // Linked to the r_brace below
#else
while(b) { // Also linked to the r_brace below
#endif
foo();
} // Linked to the l_brace of while(b) above
The `l_brace` of `while (a)` is still linked to the `r_brace` that has been
re-linked to another `l_bace`.
Because `MatchingParen` will be reset in the annotator anyway, linking it for
block braces in `UnwrappedLineParser` has little or no value. I will revert
this patch if no one objects.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139257/new/
https://reviews.llvm.org/D139257
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
