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

Optimisation to make the CheckUninitVar::parse slightly faster

This commit is contained in:
Daniel Marjamäki 2010-09-23 21:40:08 +02:00
parent d727fa858c
commit 8950beac37
1 changed files with 3 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lib/checkother.cpp
View File

@ -3189,7 +3189,7 @@ private:
return &tok; return &tok;
} }
if (Token::Match(tok.previous(), "[;{}] %var% =|[|.")) if (Token::Match(tok.previous(), "[;{}] %var% [=[.]"))
{ {
if (tok.next()->str() == ".") if (tok.next()->str() == ".")
{ {
@ -3203,7 +3203,7 @@ private:
// check variable usages in rhs/index // check variable usages in rhs/index
for (const Token *tok2 = tok.tokAt(2); tok2; tok2 = tok2->next()) for (const Token *tok2 = tok.tokAt(2); tok2; tok2 = tok2->next())
{ {
if (Token::Match(tok2, ";|)|=|?")) if (Token::Match(tok2, "[;)=?]"))
break; break;
if (Token::Match(tok2, "%var% (")) if (Token::Match(tok2, "%var% ("))
break; break;
@ -3307,7 +3307,7 @@ private:
} }
} }
if (Token::Match(&tok, "%var% (") && uvarFunctions.find(tok.str()) == uvarFunctions.end()) else if (Token::Match(&tok, "%var% (") && uvarFunctions.find(tok.str()) == uvarFunctions.end())
{ {
if (Token::simpleMatch(&tok, "sizeof (")) if (Token::simpleMatch(&tok, "sizeof ("))
return tok.next()->link(); return tok.next()->link();
@ -3480,10 +3480,6 @@ private:
const unsigned int varid2 = tok.tokAt(-2)->varId(); const unsigned int varid2 = tok.tokAt(-2)->varId();
if (varid2) if (varid2)
{ {
/*
const Token *tok2 = Token::findmatch(owner->_tokenizer->tokens(), "%varid%", varid2);
if (tok2 && !Token::simpleMatch(tok2->previous(), "*"))
*/
{ {
use(checks, &tok); use(checks, &tok);
return &tok; return &tok;