From 72356f947b096fc4cc89f1b07ce5ee55047813b0 Mon Sep 17 00:00:00 2001
From: Noel Grandin <noel.grandin@collabora.co.uk>
Date: Fri, 17 Feb 2017 09:35:02 +0200
Subject: teach loplugin unusedenumconstants about CXXOperatorCallExpr

fixes some false+ in the write-only analysis

Change-Id: Ie7905c05c8012820c640064d451b33badb04c5fc
---
 compilerplugins/clang/unusedenumconstants.cxx | 7 +++++++
 1 file changed, 7 insertions(+)

(limited to 'compilerplugins')

diff --git a/compilerplugins/clang/unusedenumconstants.cxx b/compilerplugins/clang/unusedenumconstants.cxx
index 2500451b803a..8ba5d23f8485 100644
--- a/compilerplugins/clang/unusedenumconstants.cxx
+++ b/compilerplugins/clang/unusedenumconstants.cxx
@@ -164,6 +164,13 @@ try_again:
         } else {
             bRead = true;
         }
+    } else if (const CXXOperatorCallExpr * operatorCall = dyn_cast<CXXOperatorCallExpr>(parent))
+    {
+        if (operatorCall->isAssignmentOp()) {
+            bWrite = true;
+        } else {
+            bRead = true;
+        }
     } else if (isa<CastExpr>(parent) || isa<UnaryOperator>(parent)
                 || isa<ConditionalOperator>(parent) || isa<ParenExpr>(parent)
                 || isa<MaterializeTemporaryExpr>(parent)
-- 
cgit