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

Anton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 1 01:37:52 PST 2017


xgsa 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); };
----------------
JonasToth wrote:
> 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?
There is such test case (for class "B") or did you mean something else?


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D40671





More information about the cfe-commits mailing list