[PATCH] D40671: [clang-tidy] Support specific categories for NOLINT directive

Anton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 1 00:40:28 PST 2017


xgsa added a comment.

In https://reviews.llvm.org/D40671#941676, @JonasToth wrote:

> Could you please explain what category means? Could i disable all of `cppcoreguidelines` with something like `// NOLINT (cppcoreguidelines-*)`?


No, only individual checks can be disabled. You are right, by "categories" I meant just "checks". Sorry for confusion. I'll update the description.



================
Comment at: test/clang-tidy/nolintnextline.cpp:14
+
+// NOLINTNEXTLINE(not-closed-bracket-is-treated-as-skip-all
+class C2 { C2(int i); };
----------------
JonasToth wrote:
> missing `)`
No, it's intentionally: it's a test for case when `)` is missing


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D40671





More information about the cfe-commits mailing list