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

Jonas Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Dec 2 05:40:50 PST 2017


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





More information about the cfe-commits mailing list