owenpan added a comment.
@gedare thanks for changing `EXPECT_EQ` to `verifyFormat`, but IMO we should do
that in another patch so that it would be easier (at least for me) to review
the new tests and to make sure no existing tests have been changed by accident.
================
Comment at: clang/lib/Format/TokenAnnotator.cpp:5007
return false;
}
const FormatToken *Previous = Right.MatchingParen->Previous;
----------------
Can we simply add the above and leave the original `return` statement on line
5009 unchanged?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D137762/new/
https://reviews.llvm.org/D137762
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
