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

Reply via email to