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

Reverted some changes in commits 4b05c3e19f and 4b08b13845.

This commit is contained in:
Edoardo Prezioso 2012-01-16 10:51:13 +01:00
parent 4b05c3e19f
commit 40739eb3fb
1 changed files with 11 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
lib/tokenize.cpp
View File

@ -4167,9 +4167,8 @@ bool Tokenizer::simplifyIfAddBraces()
}
else if (tok->str() == "else") {
const Token *tok2 = tok->next();
// An else followed by an if or brace don't need to be processed further
if (tok2 && (tok2->str() == "if" || tok2->str() == "{"))
if (Token::Match(tok, "else if|{"))
continue;
}
@ -4214,17 +4213,12 @@ bool Tokenizer::simplifyIfAddBraces()
break;
}
tempToken = tempToken->link();
const Token *tempToken1 = tempToken->next();
if (!tempToken1)
if (!tempToken || !tempToken->next())
break;
if (Token::simpleMatch(tempToken, "} else") && !Token::Match(tempToken->tokAt(2), "if|{"))
innerIf = false;
else if (tempToken->next()->isName() && tempToken->next()->str() != "else")
break;
if (tempToken1->isName()) {
if (tempToken1->str() == "else") {
tempToken1 = tempToken1->next();
if (!tempToken1 || (tempToken1->str() != "if" && tempToken1->str() != "{"))
innerIf = false;
} else
break;
}
continue;
}
@ -4243,12 +4237,11 @@ bool Tokenizer::simplifyIfAddBraces()
if (!innerIf)
break;
const Token *tempToken1 = tempToken->next();
if (tempToken1 && tempToken1->str() == "else") {
tempToken1 = tempToken1->next();
if (!tempToken1 || tempToken1->str() != "if")
innerIf = false;
} else
if (Token::simpleMatch(tempToken, "; else if"))
;
else if (Token::simpleMatch(tempToken, "; else"))
innerIf = false;
else
break;
}
}