JonasToth added a comment. Could you please explain what category means? Could i disable all of `cppcoreguidelines` with something like `// NOLINT (cppcoreguidelines-*)`?
================ Comment at: clang-tidy/ClangTidyDiagnosticConsumer.cpp:297 + if (NolintIndex != StringRef::npos) { + auto BracketIndex = NolintIndex + NolintMacro.size(); + if (BracketIndex < Line.size() && Line[BracketIndex] == '(') { ---------------- Maybe some comments whats happening might be helpfull. It would simplify later modifications. ================ Comment at: test/clang-tidy/nolintnextline.cpp:14 + +// NOLINTNEXTLINE(not-closed-bracket-is-treated-as-skip-all +class C2 { C2(int i); }; ---------------- missing `)` Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D40671 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits