[Libreoffice-commits] core.git: compilerplugins/clang

[Stephan Bergmann]

 compilerplugins/clang/singlevalfields.cxx |    4 ----
 1 file changed, 4 deletions(-)

New commits:
commit 20f58e790441bf62bd9e3ed575a1d97f7144e85c
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Thu Jun 23 15:02:01 2016 +0200

    -Werror=unused-but-set-variable
    
    Change-Id: I0b3586bb3acca4075b26790a8dfe3f97d6921feb

diff --git a/compilerplugins/clang/singlevalfields.cxx b/compilerplugins/clang/singlevalfields.cxx
index b1c7cc8..368b767 100644
--- a/compilerplugins/clang/singlevalfields.cxx
+++ b/compilerplugins/clang/singlevalfields.cxx
@@ -278,7 +278,6 @@ bool SingleValFields::VisitMemberExpr( const MemberExpr* memberExpr )
             if (!functionDecl) {
                 break;
             }
-            bool bFound = false;
             for (unsigned i = 0; i < callExpr->getNumArgs(); ++i) {
                 if (i >= functionDecl->getNumParams()) // can happen in template code
                     break;
@@ -289,7 +288,6 @@ bool SingleValFields::VisitMemberExpr( const MemberExpr* memberExpr )
                         assignValue = "?";
                         bPotentiallyAssignedTo = true;
                     }
-                    bFound = true;
                     break;
                 }
             }
@@ -299,7 +297,6 @@ bool SingleValFields::VisitMemberExpr( const MemberExpr* memberExpr )
         {
             const CXXConstructExpr* consExpr = dyn_cast<CXXConstructExpr>(parent);
             const CXXConstructorDecl* consDecl = consExpr->getConstructor();
-            bool bFound = false;
             for (unsigned i = 0; i < consExpr->getNumArgs(); ++i) {
                 if (i >= consDecl->getNumParams()) // can happen in template code
                     break;
@@ -310,7 +307,6 @@ bool SingleValFields::VisitMemberExpr( const MemberExpr* memberExpr )
                         assignValue = "?";
                         bPotentiallyAssignedTo = true;
                     }
-                    bFound = true;
                     break;
                 }
             }