[PATCH] D111208: [clang-tidy] Support globbing in NOLINT* expressions

Salman Javed via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 12 04:09:26 PDT 2021


salman-javed-nz added inline comments.


================
Comment at: clang-tools-extra/docs/clang-tidy/index.rst:314
+are ignored here, as they would effectively re-activate the warning, causing
+confusion to the users.
 
----------------
There's something about this "causing confusion to the users" that my mind latches on to. I'm not sure what my problem with it is. Perhaps the sentence will read better if you just remove the 2nd half? 


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

https://reviews.llvm.org/D111208



More information about the llvm-commits mailing list