commit cd01a470e438ac3f25f561221ef72e29cf8e1df8
author Noel Grandin <noel@peralex.com> Sun Jun 26 11:50:48 2016 +0200
committer Noel Grandin <noel@peralex.com> Sun Jun 26 11:51:34 2016 +0200
tree c260c361200606619b6cb09b2488a0a77706fac7
parent ff25ea3d5ccf3a990767cbb1ef99037d3f84b072

dont ignore destructors

generate more false+ than it helps to find interesting stuff

Change-Id: I752f08a28e629bce00264108399b2a4c8afd388f

compilerplugins/clang/singlevalfields.cxx

diff --git a/compilerplugins/clang/singlevalfields.cxx b/compilerplugins/clang/singlevalfields.cxx
index 688b0df..9fa9293 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