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

[Noel Grandin (via logerrit)]

 compilerplugins/clang/sequentialassign.cxx |    7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

New commits:
commit 80cb1ecb1863dedbb9db59a872af2d6a8dfd833d
Author:     Noel Grandin <noel.grandin at collabora.co.uk>
AuthorDate: Fri Apr 12 17:52:44 2019 +0200
Commit:     Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Fri Apr 12 17:52:44 2019 +0200

    -Werror=unused-param
    
    Change-Id: I066b74763e45c785a9ebd7094fc33f44b8ddefb6

diff --git a/compilerplugins/clang/sequentialassign.cxx b/compilerplugins/clang/sequentialassign.cxx
index 7774ce887b12..f2faa4b25150 100644
--- a/compilerplugins/clang/sequentialassign.cxx
+++ b/compilerplugins/clang/sequentialassign.cxx
@@ -162,7 +162,7 @@ private:
     const VarDecl* findSimpleAssign(Stmt const*);
     bool isSimpleRHS(Expr const*);
     Expr const* ignore(Expr const*);
-    void checkForSecondAssign(Stmt const* firstStmt, Stmt const* stmt, VarDecl const* varDecl);
+    void checkForSecondAssign(Stmt const* stmt, VarDecl const* varDecl);
 };
 
 bool SequentialAssign::VisitCompoundStmt(CompoundStmt const* compoundStmt)
@@ -186,14 +186,13 @@ bool SequentialAssign::VisitCompoundStmt(CompoundStmt const* compoundStmt)
         ++it;
         if (it == compoundStmt->body_end())
             break;
-        checkForSecondAssign(firstStmt, *it, foundVars);
+        checkForSecondAssign(*it, foundVars);
     }
 
     return true;
 }
 
-void SequentialAssign::checkForSecondAssign(Stmt const* firstStmt, Stmt const* stmt,
-                                            VarDecl const* varDecl)
+void SequentialAssign::checkForSecondAssign(Stmt const* stmt, VarDecl const* varDecl)
 {
     if (auto exprCleanup = dyn_cast<ExprWithCleanups>(stmt))
         stmt = exprCleanup->getSubExpr();