diff options
| author | Benjamin Kramer <benny.kra@googlemail.com> | 2014-09-03 13:30:28 +0000 |
|---|---|---|
| committer | Benjamin Kramer <benny.kra@googlemail.com> | 2014-09-03 13:30:28 +0000 |
| commit | 6bcbe6e04d0ff61e3eec9eb01d3c535fd2f78a38 (patch) | |
| tree | 09812252794454f36e99d0d56827f2bef1d6bcb2 /clang-tools-extra/clang-tidy/misc | |
| parent | b7f59d6bbd6c97b6d35467fcce635fc4884d230b (diff) | |
| download | bcm5719-llvm-6bcbe6e04d0ff61e3eec9eb01d3c535fd2f78a38.tar.gz bcm5719-llvm-6bcbe6e04d0ff61e3eec9eb01d3c535fd2f78a38.zip | |
[clang-tidy] Eliminate inline copies of InTemplateInstantiation matcher.
llvm-svn: 217036
Diffstat (limited to 'clang-tools-extra/clang-tidy/misc')
| -rw-r--r-- | clang-tools-extra/clang-tidy/misc/UndelegatedConstructor.cpp | 4 | ||||
| -rw-r--r-- | clang-tools-extra/clang-tidy/misc/UnusedRAII.cpp | 13 |
2 files changed, 6 insertions, 11 deletions
diff --git a/clang-tools-extra/clang-tidy/misc/UndelegatedConstructor.cpp b/clang-tools-extra/clang-tidy/misc/UndelegatedConstructor.cpp index 7e31e1bd9a7..143c27a98b0 100644 --- a/clang-tools-extra/clang-tidy/misc/UndelegatedConstructor.cpp +++ b/clang-tools-extra/clang-tidy/misc/UndelegatedConstructor.cpp @@ -60,9 +60,7 @@ void UndelegatedConstructorCheck::registerMatchers(MatchFinder *Finder) { constructExpr(hasDeclaration(constructorDecl(ofClass( recordDecl(baseOfBoundNode("parent")))))) .bind("construct"))), - unless(hasAncestor(decl( - anyOf(recordDecl(ast_matchers::isTemplateInstantiation()), - functionDecl(ast_matchers::isTemplateInstantiation())))))), + unless(isInTemplateInstantiation())), this); } diff --git a/clang-tools-extra/clang-tidy/misc/UnusedRAII.cpp b/clang-tools-extra/clang-tidy/misc/UnusedRAII.cpp index 1aabf6a10ba..41327b65bb5 100644 --- a/clang-tools-extra/clang-tidy/misc/UnusedRAII.cpp +++ b/clang-tools-extra/clang-tidy/misc/UnusedRAII.cpp @@ -29,14 +29,11 @@ void UnusedRAIICheck::registerMatchers(MatchFinder *Finder) { // those returned from a call. auto BindTemp = bindTemporaryExpr(unless(has(callExpr()))).bind("temp"); Finder->addMatcher( - exprWithCleanups( - unless(hasAncestor(decl( - anyOf(recordDecl(ast_matchers::isTemplateInstantiation()), - functionDecl(ast_matchers::isTemplateInstantiation()))))), - hasParent(compoundStmt().bind("compound")), - hasType(recordDecl(hasUserDeclaredDestructor())), - anyOf(has(BindTemp), has(functionalCastExpr(has(BindTemp))))) - .bind("expr"), + exprWithCleanups(unless(isInTemplateInstantiation()), + hasParent(compoundStmt().bind("compound")), + hasType(recordDecl(hasUserDeclaredDestructor())), + anyOf(has(BindTemp), has(functionalCastExpr( + has(BindTemp))))).bind("expr"), this); } |

