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

[Jonas Toth via Phabricator via cfe-commits]

JonasToth added a comment.

Its good that you added code examples to the clang-tidy documentation page. I think that feature was not documented well before.



================
Comment at: test/clang-tidy/nolintnextline.cpp:23
+
+// NOLINTNEXTLINE without-brackets-skip-all, another-check
+class C5 { C5(int i); };
----------------
Ian confused now. The NOLINTNEXTLINE with incorrect parents should not silence the diagnostic, should it? 

In my understanding the following line should cause the explicit constructor check to warn. Is that check message missing or did I get something wrong?


https://reviews.llvm.org/D40671