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

Refactoring: Use %op% instead of Token::isOp in CheckBufferOverrun

This commit is contained in:
Daniel Marjamäki 2011-04-09 18:37:03 +02:00
parent a4471bc150
commit bd93997e4f
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/checkbufferoverrun.cpp
View File

@ -707,7 +707,7 @@ void CheckBufferOverrun::checkFunctionParameter(const Token &tok, unsigned int p
Token::Match(ftok, "%var% --")) Token::Match(ftok, "%var% --"))
break; break;
if ((Token::Match(ftok->previous(), "[=;{}]") || ftok->previous()->isOp()) && Token::Match(ftok, "%var% [ %num% ]")) if (Token::Match(ftok->previous(), "=|;|{|}|%op% %var% [ %num% ]"))
{ {
const MathLib::bigint index = MathLib::toLongNumber(ftok->strAt(2)); const MathLib::bigint index = MathLib::toLongNumber(ftok->strAt(2));
if (index >= 0 && arrayInfo.num[0] > 0 && index >= arrayInfo.num[0]) if (index >= 0 && arrayInfo.num[0] > 0 && index >= arrayInfo.num[0])