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

Jonas Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 1 01:26:09 PST 2017


JonasToth added inline comments.


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

A testcase for where all parens are missing would be nice, too. I assume that everything will be ignored, is that correct?


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D40671





More information about the cfe-commits mailing list