diff --git a/lib/checkclass.cpp b/lib/checkclass.cpp index 8298cdb75..c9fcd38e3 100644 --- a/lib/checkclass.cpp +++ b/lib/checkclass.cpp @@ -890,7 +890,7 @@ static bool checkFunctionUsage(const Function *privfunc, const Scope* scope) for (const Token *ftok = func->functionScope->classDef->linkAt(1); ftok != func->functionScope->classEnd; ftok = ftok->next()) { if (ftok->function() == privfunc) return true; - if (ftok->varId() == 0U && !ftok->function() && ftok->str() == privfunc->name()) // TODO: This condition should be redundant + if (ftok->varId() == 0U && ftok->str() == privfunc->name()) // TODO: This condition should be redundant return true; } } else if ((func->type != Function::eCopyConstructor && diff --git a/test/testunusedprivfunc.cpp b/test/testunusedprivfunc.cpp index 988ff3e19..85b1d0314 100644 --- a/test/testunusedprivfunc.cpp +++ b/test/testunusedprivfunc.cpp @@ -408,7 +408,7 @@ private: " void f() { }\n" " void f(int) { }\n" "};"); - ASSERT_EQUALS("[test.cpp:10]: (style) Unused private function: 'A::f'\n", errout.str()); + ASSERT_EQUALS("", errout.str()); } void incompleteImplementation() {