diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-04-12 17:52:44 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-04-12 17:52:44 +0200 |
commit | 80cb1ecb1863dedbb9db59a872af2d6a8dfd833d (patch) | |
tree | 6a0cac93b2b39286baf13378f1677b5b750c6521 | |
parent | 9769288a83ba038e8dbbb16261436a1cd53b37a1 (diff) |
-Werror=unused-param
Change-Id: I066b74763e45c785a9ebd7094fc33f44b8ddefb6
-rw-r--r-- | compilerplugins/clang/sequentialassign.cxx | 7 |
1 files changed, 3 insertions, 4 deletions
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(); |