[PATCH] D53771: [clang-tidy] Avoid C arrays check

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Nov 10 07:43:39 PST 2018


lebedev.ri added a comment.

*Maybe* we should be actually diagnosing the each actual declaration, not just the `typeloc`.
Else if you use one `typedef`, and `// NOLINT` it, you will silence it for all the decls that use it.

In https://reviews.llvm.org/D53771#1294241, @JonasToth wrote:

> LGTM.
>  Please give other reviewers a chance to take a look at it too.


Thank you for the review!
I will keep this open for a few more days.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53771





More information about the cfe-commits mailing list