diff options
author | Haojian Wu <hokein@google.com> | 2016-02-03 12:10:27 +0000 |
---|---|---|
committer | Haojian Wu <hokein@google.com> | 2016-02-03 12:10:27 +0000 |
commit | 29634fe676c855777562f8697ebfc3bb6edd5c64 (patch) | |
tree | e523b91c30017116c173032f15b7202de77a656a /clang-tools-extra/clang-tidy/misc/DefinitionsInHeadersCheck.cpp | |
parent | 5f496fb941fca3264c08c8559f6abbc940cfcfb2 (diff) | |
download | bcm5719-llvm-29634fe676c855777562f8697ebfc3bb6edd5c64.tar.gz bcm5719-llvm-29634fe676c855777562f8697ebfc3bb6edd5c64.zip |
[clang-tidy] bug fix: Don't warn on partial template specialization in `misc-definitions-in-headers` check.
Reviewers: alexfh
Subscribers: cfe-commits
Differential Revision: http://reviews.llvm.org/D16578
llvm-svn: 259643
Diffstat (limited to 'clang-tools-extra/clang-tidy/misc/DefinitionsInHeadersCheck.cpp')
-rw-r--r-- | clang-tools-extra/clang-tidy/misc/DefinitionsInHeadersCheck.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/clang-tools-extra/clang-tidy/misc/DefinitionsInHeadersCheck.cpp b/clang-tools-extra/clang-tidy/misc/DefinitionsInHeadersCheck.cpp index 29fa0f6d5bf..1b9cf29c041 100644 --- a/clang-tools-extra/clang-tidy/misc/DefinitionsInHeadersCheck.cpp +++ b/clang-tools-extra/clang-tidy/misc/DefinitionsInHeadersCheck.cpp @@ -91,9 +91,12 @@ void DefinitionsInHeadersCheck::check(const MatchFinder::MatchResult &Result) { if (const auto *MD = dyn_cast<CXXMethodDecl>(FD)) { const auto *DC = MD->getDeclContext(); while (DC->isRecord()) { - if (const auto *RD = dyn_cast<CXXRecordDecl>(DC)) + if (const auto *RD = dyn_cast<CXXRecordDecl>(DC)) { + if (isa<ClassTemplatePartialSpecializationDecl>(RD)) + return; if (RD->getDescribedClassTemplate()) return; + } DC = DC->getParent(); } } |