walkero/cppcheck
1
0
Fork 0
Code Issues Pull Requests Projects Releases 1 Wiki Activity

Combined consecutive Token::simpleMatch() calls

Browse Source
This commit is contained in:
orbitcowboy 2022-06-02 07:27:19 +02:00
parent 3b2509c9cc
commit 7275350f3e
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/checkfunctions.cpp
View File

@ -144,12 +144,12 @@ void CheckFunctions::invalidFunctionUsage()
auto count = -1; // Find out explicitly set count, e.g.: char buf[3] = {...}. Variable 'count' is set to 3 then. auto count = -1; // Find out explicitly set count, e.g.: char buf[3] = {...}. Variable 'count' is set to 3 then.
if (varTok && Token::simpleMatch(varTok->previous(), "]")) if (varTok && Token::simpleMatch(varTok->previous(), "]"))
{ {
const Token* countTok = varTok->tokAt(-2); const Token* const countTok = varTok->tokAt(-2);
if (countTok && countTok->hasKnownIntValue()) if (countTok && countTok->hasKnownIntValue())
count = countTok->getKnownIntValue(); count = countTok->getKnownIntValue();
} }
if (Token::simpleMatch(varTok, "=") && Token::simpleMatch(varTok->next(), "{")) { if (Token::simpleMatch(varTok, "= {")) {
varTok = varTok->next(); varTok = varTok->tokAt(1);
auto actualCharCount = 0; auto actualCharCount = 0;
while (varTok && !Token::simpleMatch(varTok->next(), "}")) { while (varTok && !Token::simpleMatch(varTok->next(), "}")) {
if (!Token::simpleMatch(varTok->next(), ",")) if (!Token::simpleMatch(varTok->next(), ","))