[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