[PATCH] D116378: [clang-tidy] Disable clang-tidy warnings from system macros

Carlos Galvez via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 6 12:23:37 PST 2022


carlosgalvezp updated this revision to Diff 397958.
carlosgalvezp added a comment.

- Refactor duplicated warning message into a common variable.
- Remove double backticks change, TBH I get conflicting review comments about this and can't find any docs supporting either, so I'll just keep it consistent.
- Move extra class to the existing anonymous namespace instead of creating a new one.


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

https://reviews.llvm.org/D116378

Files:
  clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.cpp
  clang-tools-extra/clang-tidy/cppcoreguidelines/ProTypeVarargCheck.cpp
  clang-tools-extra/clang-tidy/cppcoreguidelines/ProTypeVarargCheck.h
  clang-tools-extra/docs/ReleaseNotes.rst
  clang-tools-extra/test/clang-tidy/infrastructure/Inputs/file-filter/system/system-header.h
  clang-tools-extra/test/clang-tidy/infrastructure/file-filter.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D116378.397958.patch
Type: text/x-patch
Size: 6674 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220106/a944a67e/attachment-0001.bin>


More information about the cfe-commits mailing list