diff options
author | Noel Grandin <noel@peralex.com> | 2016-06-26 11:50:48 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2016-06-26 11:51:34 +0200 |
commit | cd01a470e438ac3f25f561221ef72e29cf8e1df8 (patch) | |
tree | c260c361200606619b6cb09b2488a0a77706fac7 | |
parent | ff25ea3d5ccf3a990767cbb1ef99037d3f84b072 (diff) |
dont ignore destructors
generate more false+ than it helps to find interesting stuff
Change-Id: I752f08a28e629bce00264108399b2a4c8afd388f
-rw-r--r-- | compilerplugins/clang/singlevalfields.cxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/compilerplugins/clang/singlevalfields.cxx b/compilerplugins/clang/singlevalfields.cxx index 688b0df8f81b..9fa9293a7b7e 100644 --- a/compilerplugins/clang/singlevalfields.cxx +++ b/compilerplugins/clang/singlevalfields.cxx @@ -191,8 +191,7 @@ bool SingleValFields::VisitMemberExpr( const MemberExpr* memberExpr ) return true; const CXXMethodDecl* methodDecl = dyn_cast_or_null<CXXMethodDecl>(get_top_FunctionDecl_from_Stmt(*memberExpr)); - if (methodDecl && (methodDecl->isCopyAssignmentOperator() || methodDecl->isMoveAssignmentOperator() - || dyn_cast<CXXDestructorDecl>(methodDecl))) + if (methodDecl && (methodDecl->isCopyAssignmentOperator() || methodDecl->isMoveAssignmentOperator())) return true; // walk up the tree until we find something interesting |