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

Merge pull request #381 from Dmitry-Me/checkBooleanFlagFirst 

Run cheaper check first, don't check the same condition twice.
This commit is contained in:
Daniel Marjamäki 2014-07-28 18:02:49 +02:00
parent 8a1c5d6fce dc03b98e60
commit 905a383ac2
1 changed files with 9 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
lib/tokenize.cpp
View File

@ -5335,15 +5335,15 @@ void Tokenizer::simplifyVarDecl(Token * tokBegin, Token * tokEnd, bool only_k_r_
for (Token *tok3 = tok2; tok3; tok3 = tok3->next()) {
++typelen;
if (tok3->str() == "<" && !parens) {
if (!parens && tok3->str() == "<") {
++indentlevel;
} else if (tok3->str() == ">" && !parens) {
} else if (!parens && tok3->str() == ">") {
if (indentlevel == 0) {
tok2 = tok3->next();
break;
}
--indentlevel;
} else if (tok3->str() == ">>" && !parens) {
} else if (!parens && tok3->str() == ">>") {
if (indentlevel <= 1U) {
tok2 = tok3->next();
break;
@ -5488,11 +5488,12 @@ void Tokenizer::simplifyPlatformTypes()
type = isLong;
else
type = isLongLong;
} else if (_settings->sizeof_size_t == 4 && _settings->sizeof_long == 4)
} else if (_settings->sizeof_size_t == 4) {
if (_settings->sizeof_long == 4)
type = isLong;
else if (_settings->sizeof_size_t == 4)
type = isInt;
else
type = isInt;
} else
return;
for (Token *tok = list.front(); tok; tok = tok->next()) {