diff --git a/lib/forwardanalyzer.cpp b/lib/forwardanalyzer.cpp index a57e4ddd1..410d38207 100644 --- a/lib/forwardanalyzer.cpp +++ b/lib/forwardanalyzer.cpp @@ -555,7 +555,10 @@ struct ForwardTraversal { }; -ForwardAnalyzer::Action valueFlowGenericForward(Token* start, const Token* end, const ValuePtr& fa, const Settings* settings) +ForwardAnalyzer::Action valueFlowGenericForward(Token* start, + const Token* end, + const ValuePtr& fa, + const Settings* settings) { ForwardTraversal ft{fa, settings}; ft.updateRange(start, end); diff --git a/lib/forwardanalyzer.h b/lib/forwardanalyzer.h index d80ce8654..d1b1dc2c4 100644 --- a/lib/forwardanalyzer.h +++ b/lib/forwardanalyzer.h @@ -113,6 +113,9 @@ struct ForwardAnalyzer { virtual ~ForwardAnalyzer() {} }; -ForwardAnalyzer::Action valueFlowGenericForward(Token* start, const Token* end, const ValuePtr& fa, const Settings* settings); +ForwardAnalyzer::Action valueFlowGenericForward(Token* start, + const Token* end, + const ValuePtr& fa, + const Settings* settings); #endif diff --git a/lib/valueflow.cpp b/lib/valueflow.cpp index 57d1df0c3..e1545eb0c 100644 --- a/lib/valueflow.cpp +++ b/lib/valueflow.cpp @@ -1711,11 +1711,11 @@ static void valueFlowGlobalStaticVar(TokenList *tokenList, const Settings *setti } static ForwardAnalyzer::Action valueFlowForwardVariable(Token* const startToken, - const Token* const endToken, - const Variable* const var, - std::list values, - TokenList* const tokenlist, - const Settings* const settings); + const Token* const endToken, + const Variable* const var, + std::list values, + TokenList* const tokenlist, + const Settings* const settings); // Old deprecated version static void valueFlowForward(Token* startToken, @@ -2698,12 +2698,12 @@ static std::vector getAliasesFromValues(std::list values, - std::vector aliases, - TokenList* const tokenlist, - const Settings* const settings) + const Token* const endToken, + const Variable* const var, + std::list values, + std::vector aliases, + TokenList* const tokenlist, + const Settings* const settings) { ForwardAnalyzer::Action actions; for (ValueFlow::Value& v : values) { @@ -2714,13 +2714,14 @@ static ForwardAnalyzer::Action valueFlowForwardVariable(Token* const startToken, } static ForwardAnalyzer::Action valueFlowForwardVariable(Token* const startToken, - const Token* const endToken, - const Variable* const var, - std::list values, - TokenList* const tokenlist, - const Settings* const settings) + const Token* const endToken, + const Variable* const var, + std::list values, + TokenList* const tokenlist, + const Settings* const settings) { - return valueFlowForwardVariable(startToken, endToken, var, std::move(values), getAliasesFromValues(values), tokenlist, settings); + return valueFlowForwardVariable( + startToken, endToken, var, std::move(values), getAliasesFromValues(values), tokenlist, settings); } // Old deprecated version @@ -2808,11 +2809,11 @@ struct ExpressionForwardAnalyzer : SingleValueFlowForwardAnalyzer { }; static ForwardAnalyzer::Action valueFlowForwardExpression(Token* startToken, - const Token* endToken, - const Token* exprTok, - const std::list& values, - const TokenList* const tokenlist, - const Settings* settings) + const Token* endToken, + const Token* exprTok, + const std::list& values, + const TokenList* const tokenlist, + const Settings* settings) { ForwardAnalyzer::Action actions; for (const ValueFlow::Value& v : values) { @@ -2884,20 +2885,15 @@ static const Token* solveExprValues(const Token* expr, std::list values, - TokenList* const tokenlist, - const Settings* settings) + const Token* endToken, + const Token* exprTok, + std::list values, + TokenList* const tokenlist, + const Settings* settings) { const Token* expr = solveExprValues(exprTok, values); if (expr->variable()) { - return valueFlowForwardVariable(startToken, - endToken, - expr->variable(), - values, - tokenlist, - settings); + return valueFlowForwardVariable(startToken, endToken, expr->variable(), values, tokenlist, settings); } else { return valueFlowForwardExpression(startToken, endToken, expr, values, tokenlist, settings); } @@ -4553,13 +4549,10 @@ static void valueFlowAfterCondition(TokenList *tokenlist, const Settings *settings) { ValueFlowConditionHandler handler; - handler.forward = [&](Token* start, - const Token* stop, - const Token* vartok, - const std::list& values, - bool) { - return valueFlowForward(start->next(), stop, vartok, values, tokenlist, settings).isModified(); - }; + handler.forward = + [&](Token* start, const Token* stop, const Token* vartok, const std::list& values, bool) { + return valueFlowForward(start->next(), stop, vartok, values, tokenlist, settings).isModified(); + }; handler.parse = [&](const Token *tok) { ValueFlowConditionHandler::Condition cond; ValueFlow::Value true_value; @@ -5889,12 +5882,19 @@ struct ContainerVariableForwardAnalyzer : VariableForwardAnalyzer { } }; -static ForwardAnalyzer::Action valueFlowContainerForward(Token *tok, const Token* endToken, const Variable* var, ValueFlow::Value value, TokenList *tokenlist) +static ForwardAnalyzer::Action valueFlowContainerForward(Token* tok, + const Token* endToken, + const Variable* var, + ValueFlow::Value value, + TokenList* tokenlist) { ContainerVariableForwardAnalyzer a(var, value, getAliasesFromValues({value}), tokenlist); return valueFlowGenericForward(tok, endToken, a, tokenlist->getSettings()); } -static ForwardAnalyzer::Action valueFlowContainerForward(Token *tok, const Variable* var, ValueFlow::Value value, TokenList *tokenlist) +static ForwardAnalyzer::Action valueFlowContainerForward(Token* tok, + const Variable* var, + ValueFlow::Value value, + TokenList* tokenlist) { const Token * endOfVarScope = nullptr; if (var->isLocal() || var->isArgument()) @@ -6050,13 +6050,10 @@ static void valueFlowIteratorAfterCondition(TokenList *tokenlist, const Settings *settings) { ValueFlowConditionHandler handler; - handler.forward = [&](Token* start, - const Token* stop, - const Token* vartok, - const std::list& values, - bool) { - return valueFlowForward(start->next(), stop, vartok, values, tokenlist, settings).isModified(); - }; + handler.forward = + [&](Token* start, const Token* stop, const Token* vartok, const std::list& values, bool) { + return valueFlowForward(start->next(), stop, vartok, values, tokenlist, settings).isModified(); + }; handler.parse = [&](const Token *tok) { ValueFlowConditionHandler::Condition cond;