[PATCH] D108560: [clang-tidy] Add support for NOLINTBEGIN ... NOLINTEND comments to suppress clang-tidy warnings over multiple lines

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added inline comments.


================
Comment at: clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend.cpp:82
+
+// CHECK-MESSAGES: Suppressed 12 warnings (12 NOLINT)
----------------
I'd like to see some additional test coverage that shows improper use and what happens.

e.g., what happens if there's a `NOLINTBEGIN` and no `NOLINTEND` marker? What happens if `NOLINTEND` appears before any `NOLINTBEGIN`?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108560/new/

https://reviews.llvm.org/D108560