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

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 1 06:59:27 PST 2017


Eugene.Zelenko added inline comments.


================
Comment at: clang-tidy/ClangTidyDiagnosticConsumer.cpp:293
 }
-
-static bool LineIsMarkedWithNOLINT(SourceManager &SM, SourceLocation Loc) {
+static bool IsNOLINTFound(StringRef NolintMacro, StringRef Line,
+                          unsigned DiagID, const ClangTidyContext &Context) {
----------------
Please separate with empty line.


================
Comment at: clang-tidy/ClangTidyDiagnosticConsumer.cpp:323
+}
+static bool LineIsMarkedWithNOLINT(SourceManager &SM, SourceLocation Loc,
+                                   unsigned DiagID,
----------------
Please separate with empty line.


https://reviews.llvm.org/D40671





More information about the cfe-commits mailing list