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

Try to fix daca@home crashes

This commit is contained in:
Daniel Marjamäki 2019-07-29 19:05:36 +02:00
parent 4e6a228b97
commit f63011c219
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lib/checkunusedvar.cpp
View File

@ -1171,15 +1171,19 @@ void CheckUnusedVar::checkFunctionVariableUsage()
if (FwdAnalysis::isNullOperand(tok->astOperand2())) if (FwdAnalysis::isNullOperand(tok->astOperand2()))
continue; continue;
if (tok->astOperand1()->variable() && tok->astOperand1()->variable()->isReference() && tok->astOperand1()->variable()->nameToken() != tok->astOperand1()) if (!tok->astOperand1())
continue;
const Variable *op1Var = tok->astOperand1() ? tok->astOperand1()->variable() : nullptr;
if (op1Var && op1Var->isReference() && op1Var->nameToken() != tok->astOperand1())
// todo: check references // todo: check references
continue; continue;
if (tok->astOperand1()->variable() && tok->astOperand1()->variable()->isStatic()) if (op1Var && op1Var->isStatic())
// todo: check static variables // todo: check static variables
continue; continue;
if (tok->astOperand1()->variable() && tok->astOperand1()->variable()->nameToken()->isAttributeUnused()) if (op1Var && op1Var->nameToken()->isAttributeUnused())
continue; continue;
// Is there a redundant assignment? // Is there a redundant assignment?