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

Salman Javed via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Oct 10 04:15:33 PDT 2021


salman-javed-nz added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.cpp:446
     SuppressionErrors.emplace_back(Error.getValue());
-    return false;
   }
----------------
carlosgalvezp wrote:
> I had to remove these "return false", otherwise I would not get errors regarding "Found NOLINTBEGIN that was unmatched by a NOLINTEND". Not really sure why they were there in the first place.
> 
> All unit tests pass with this patch.
The reason for the `return false` was because the file is already determined to have a unmatched `NOLINTBEGIN`, so it's FUBAR and probably not worth checking the remainder of anyway. I don't mind either way. 


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

https://reviews.llvm.org/D111208



More information about the cfe-commits mailing list