This revision was automatically updated to reflect the committed changes.
Closed by commit rL295435: [clang-tidy] Add cert-dcl58-cpp (do not modify the
'std' namespace) check. (authored by xazax).
Changed prior to commit:
https://reviews.llvm.org/D23421?vs=88541&id=88863#toc
Repository:
rL L
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM, thank you for working on this!
https://reviews.llvm.org/D23421
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http:/
xazax.hun updated this revision to Diff 88541.
xazax.hun added a comment.
- Do not warn for function specializations within the std namespace.
- Add a test case for swap.
https://reviews.llvm.org/D23421
Files:
clang-tidy/cert/CERTTidyModule.cpp
clang-tidy/cert/CMakeLists.txt
clang-tidy/ce
aaron.ballman added inline comments.
Comment at: clang-tidy/cert/DontModifyStdNamespaceCheck.cpp:28
+ anyOf(hasName("std"), hasName("posix")),
+ has(decl(unless(cxxRecordDecl(isExplicitTemplateSpecialization())
+ .bind("nmspc"),
I t
xazax.hun updated this revision to Diff 88506.
xazax.hun marked 3 inline comments as done.
xazax.hun retitled this revision from "[Clang-tidy] CERT-MSC53-CPP (checker for
std namespace modification)" to "[Clang-tidy] CERT-DCL58-CPP (checker for std
namespace modification)".
xazax.hun edited the s