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

templatesimplifier: remove redundant assignments inside switchcase. They will always be overwritten by an assignment outside of the switch near the end of the function. Found by clang analyzer.

This commit is contained in:
Matthias Krüger 2017-01-30 22:26:54 +01:00
parent d846217641
commit 92d9e810f3
1 changed files with 0 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
lib/templatesimplifier.cpp
View File

@ -935,23 +935,18 @@ bool TemplateSimplifier::simplifyNumericCalculations(Token *tok)
switch (op->str()[0]) { switch (op->str()[0]) {
case '<': case '<':
tok->str((v1 << v2).str()); tok->str((v1 << v2).str());
ret = true;
break; break;
case '>': case '>':
tok->str((v1 >> v2).str()); tok->str((v1 >> v2).str());
ret = true;
break; break;
case '&': case '&':
tok->str((v1 & v2).str()); tok->str((v1 & v2).str());
ret = true;
break; break;
case '|': case '|':
tok->str((v1 | v2).str()); tok->str((v1 | v2).str());
ret = true;
break; break;
case '^': case '^':
tok->str((v1 ^ v2).str()); tok->str((v1 ^ v2).str());
ret = true;
break; break;
}; };
} }