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

Fixed ticket #228: false positive with usage of an auto-variable

This commit is contained in:
Gianluca Scacco 2009-03-28 21:47:38 +01:00
parent 6f7f8c4b4f
commit c88a362893
2 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/checkautovariables.cpp
View File

@ -147,19 +147,25 @@ void CheckAutoVariables::autoVariables()
{ {
if (Token::Match(tok, "%type% %var% (") || if (Token::Match(tok, "%type% %var% (") ||
Token::Match(tok, "%type% * %var% (") || Token::Match(tok, "%type% * %var% (") ||
Token::Match(tok, "%type% :: %var% (")) Token::Match(tok, "%type% :: %var% ("))
{ {
begin_function = true; begin_function = true;
fp_list.clear(); fp_list.clear();
vd_list.clear(); vd_list.clear();
} }
else if (begin_function && begin_function_decl && Token::Match(tok, "%type% * %var%")) else if (begin_function && begin_function_decl && Token::Match(tok, "%type% * * %var%"))
{
std::string var_name;
var_name = tok->tokAt(3)->str();
fp_list.push_back(var_name);
}
else if (begin_function && begin_function_decl && Token::Match(tok, "%type% * %var% ["))
{ {
std::string var_name; std::string var_name;
var_name = tok->tokAt(2)->str(); var_name = tok->tokAt(2)->str();
//cout << "FP " << tok->linenr() << " " << var_name << endl;
fp_list.push_back(var_name); fp_list.push_back(var_name);
} }
else if (begin_function && Token::Match(tok, "(")) else if (begin_function && Token::Match(tok, "("))

2
test/testautovariables.cpp
View File

@ -70,7 +70,7 @@ private:
void testautovar() void testautovar()
{ {
check("void func1(int *res)\n" check("void func1(int **res)\n"
"{\n" "{\n"
" int num=2;" " int num=2;"
"res=&num;"); "res=&num;");