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

Fixed #1522 (false positive: function can be const (assignment in return))

This commit is contained in:
Robert Reif 2010-03-24 19:31:30 +01:00 committed by Daniel Marjamäki
parent bef1857221
commit e95bc41e59
2 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/checkclass.cpp
View File

@ -1743,7 +1743,7 @@ bool CheckClass::sameFunc(int nest, const Token *firstEnd, const Token *secondEn
bool CheckClass::isMemberVar(const Var *varlist, const Token *tok) bool CheckClass::isMemberVar(const Var *varlist, const Token *tok)
{ {
while (tok->previous() && !Token::Match(tok->previous(), "}|{|;|public:|protected:|private:")) while (tok->previous() && !Token::Match(tok->previous(), "}|{|;|public:|protected:|private:|return"))
{ {
if (Token::Match(tok->previous(), "* this")) if (Token::Match(tok->previous(), "* this"))
return true; return true;

12
test/testclass.cpp
View File

@ -94,6 +94,7 @@ private:
TEST_CASE(const7); TEST_CASE(const7);
TEST_CASE(const8); // ticket #1517 TEST_CASE(const8); // ticket #1517
TEST_CASE(const9); // ticket #1515 TEST_CASE(const9); // ticket #1515
TEST_CASE(const10);
TEST_CASE(constoperator); // operator< can often be const TEST_CASE(constoperator); // operator< can often be const
TEST_CASE(constincdec); // increment/decrement => non-const TEST_CASE(constincdec); // increment/decrement => non-const
TEST_CASE(constReturnReference); TEST_CASE(constReturnReference);
@ -2155,6 +2156,17 @@ private:
ASSERT_EQUALS("", errout.str()); ASSERT_EQUALS("", errout.str());
} }
void const10()
{
checkConst("class A {\n"
"public:\n"
" int foo() { return x = 0; }\n"
"private:\n"
" int x;\n"
"}");
ASSERT_EQUALS("", errout.str());
}
// increment/decrement => not const // increment/decrement => not const
void constincdec() void constincdec()
{ {