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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
