From 50fb9cea6608e19c6afaeaac733daceedd139d35 Mon Sep 17 00:00:00 2001 From: Dmitry-Me Date: Tue, 11 Aug 2015 17:20:30 +0300 Subject: [PATCH] Reduce variables scope --- lib/valueflow.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/valueflow.cpp b/lib/valueflow.cpp index 73bb648aa..7eb6e3549 100644 --- a/lib/valueflow.cpp +++ b/lib/valueflow.cpp @@ -1354,9 +1354,6 @@ static bool valueFlowForward(Token * const startToken, // increment/decrement if (Token::Match(tok2->previous(), "++|-- %name%") || Token::Match(tok2, "%name% ++|--")) { - const bool pre = Token::Match(tok2->previous(), "++|--"); - Token * const op = pre ? tok2->previous() : tok2->next(); - const bool inc = (op->str() == "++"); std::list::iterator it; // Erase values that are not int values.. for (it = values.begin(); it != values.end();) { @@ -1370,6 +1367,9 @@ static bool valueFlowForward(Token * const startToken, bailout(tokenlist, errorLogger, tok2, "increment/decrement of " + tok2->str()); return false; } + const bool pre = Token::Match(tok2->previous(), "++|--"); + Token * const op = pre ? tok2->previous() : tok2->next(); + const bool inc = (op->str() == "++"); // Perform increment/decrement.. for (it = values.begin(); it != values.end(); ++it) { if (!pre)