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

CheckOther::functionVariableUsage: minor optimization.

This commit is contained in:
Nicolas Le Cam 2009-01-20 22:40:10 +00:00
parent 5eb89414c8
commit acdc381ff3
1 changed files with 11 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
src/checkother.cpp
View File

@ -867,7 +867,7 @@ void CheckOther::functionVariableUsage()
static const unsigned int USAGE_WRITE = 4; static const unsigned int USAGE_WRITE = 4;
int indentlevel = 0; int indentlevel = 0;
for (const Token *tok = tok1; tok; tok = tok->next()) for (const Token *tok = tok1->next(); tok; tok = tok->next())
{ {
if (tok->str() == "{") if (tok->str() == "{")
++indentlevel; ++indentlevel;
@ -902,34 +902,34 @@ void CheckOther::functionVariableUsage()
if (Token::Match(tok, "[;{}] bool|char|short|int|long|float|double %var% ;|=")) if (Token::Match(tok, "[;{}] bool|char|short|int|long|float|double %var% ;|="))
varUsage[ tok->strAt(2)] = USAGE_DECLARE; varUsage[ tok->strAt(2)] = USAGE_DECLARE;
if (Token::Match(tok, "[;{}] bool|char|short|int|long|float|double * %var% ;|=")) else if (Token::Match(tok, "[;{}] bool|char|short|int|long|float|double * %var% ;|="))
varUsage[ tok->strAt(3)] = USAGE_DECLARE; varUsage[ tok->strAt(3)] = USAGE_DECLARE;
if (Token::Match(tok, "delete|return %var%")) else if (Token::Match(tok, "delete|return %var%"))
varUsage[ tok->strAt(1)] |= USAGE_READ; varUsage[ tok->strAt(1)] |= USAGE_READ;
if (Token::Match(tok, "%var% =")) else if (Token::Match(tok, "%var% ="))
varUsage[ tok->str()] |= USAGE_WRITE; varUsage[ tok->str()] |= USAGE_WRITE;
if (Token::Match(tok, "else %var% =")) else if (Token::Match(tok, "else %var% ="))
varUsage[ tok->strAt(1)] |= USAGE_WRITE; varUsage[ tok->strAt(1)] |= USAGE_WRITE;
if (Token::Match(tok, ">>|& %var%")) else if (Token::Match(tok, ">>|& %var%"))
varUsage[ tok->strAt(1)] |= (USAGE_WRITE | USAGE_READ); varUsage[ tok->strAt(1)] |= (USAGE_WRITE | USAGE_READ);
if ((Token::Match(tok, "[(=&!]") || isOp(tok)) && Token::Match(tok->next(), "%var%")) else if ((Token::Match(tok, "[(=&!]") || isOp(tok)) && Token::Match(tok->next(), "%var%"))
varUsage[ tok->strAt(1)] |= USAGE_READ; varUsage[ tok->strAt(1)] |= USAGE_READ;
if (Token::Match(tok, "-=|+=|*=|/=|&=|^= %var%") || Token::Match(tok, "|= %var%")) else if (Token::Match(tok, "-=|+=|*=|/=|&=|^= %var%") || Token::Match(tok, "|= %var%"))
varUsage[ tok->strAt(1)] |= USAGE_READ; varUsage[ tok->strAt(1)] |= USAGE_READ;
if (Token::Match(tok, "%var%") && (tok->next()->str() == ")" || isOp(tok->next()))) else if (Token::Match(tok, "%var%") && (tok->next()->str() == ")" || isOp(tok->next())))
varUsage[ tok->str()] |= USAGE_READ; varUsage[ tok->str()] |= USAGE_READ;
if (Token::Match(tok, "[(,] %var% [,)]")) else if (Token::Match(tok, "[(,] %var% [,)]"))
varUsage[ tok->strAt(1)] |= USAGE_WRITE; varUsage[ tok->strAt(1)] |= USAGE_WRITE;
if (Token::Match(tok, "; %var% ;")) else if (Token::Match(tok, "; %var% ;"))
varUsage[ tok->strAt(1)] |= USAGE_READ; varUsage[ tok->strAt(1)] |= USAGE_READ;
} }