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

reverted [1282] because some changes were committed by mistake

This commit is contained in:
Daniel Marjamäki 2009-02-24 17:49:43 +00:00
parent 8c3e3faa07
commit 66c13c7173
2 changed files with 3 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/checkclass.cpp
View File

@ -476,7 +476,7 @@ void CheckClass::privateFunctions()
// The class implementation must be available..
const std::string classconstructor(classname + " :: " + classname);
if (tok1->fileIndex()>0 && !Token::findmatch(_tokenizer->tokens(), classconstructor.c_str()))
if (!Token::findmatch(_tokenizer->tokens(), classconstructor.c_str()))
continue;
// Get private functions..
@ -549,13 +549,13 @@ void CheckClass::privateFunctions()
{
if (ftok->str() == "{")
++indent_level;
else if (ftok->str() == "}")
if (ftok->str() == "}")
{
if (indent_level <= 1)
break;
--indent_level;
}
else if (Token::Match(ftok, "%var% ("))
if (Token::Match(ftok->next(), "("))
{
// Remove function from FuncList
for (std::list<const Token *>::iterator it = FuncList.begin(); it != FuncList.end(); ++it)

21
test/testunusedprivfunc.cpp
View File

@ -36,7 +36,6 @@ private:
void run()
{
TEST_CASE(test1);
TEST_CASE(test2);
// [ 2236547 ] False positive --style unused function, called via pointer
TEST_CASE(func_pointer);
@ -82,26 +81,6 @@ private:
}
void test2()
{
check("class A {\n"
"public:\n"
" A();\n"
"\n"
" void a() const\n"
" { b(); }\n"
"private:\n"
" void b( ) const\n"
" { }\n"
"};\n"
"\n"
"A::A()\n"
"{ }\n");
ASSERT_EQUALS(std::string(""), errout.str());
}