[PATCH] D90180: [clang-tidy] find/fix unneeded semicolon after switch
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 27 07:05:37 PDT 2020
aaron.ballman added a comment.
In D90180#2354931 <https://reviews.llvm.org/D90180#2354931>, @njames93 wrote:
> Is this not already handled by `-Wextra-semi`. If it isn't I feel like it should be handled there rather than in clang-tidy
Strong +1 -- this is already handled by `-Wextra-semi-stmt` and should not be duplicated in a clang-tidy check. In fact, there's already a fix-it produced by Clang for this: https://godbolt.org/z/xhbfc9
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D90180/new/
https://reviews.llvm.org/D90180
More information about the cfe-commits
mailing list