[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