Fix FP in constVariable

danmar · danmar · commit 6f4ce486a219 · 2021-09-11T20:25:46.000+02:00
diff --git a/lib/astutils.cpp b/lib/astutils.cpp
@@ -1903,7 +1903,7 @@ bool isVariableChangedByFunctionCall(const Token *tok, int indirect, const Setti
         return false; // not a function => variable not changed
     if (Token::simpleMatch(tok, "{") && isTrivialConstructor(tok))
         return false;
-    if (tok->isKeyword() && !isCPPCastKeyword(tok))
+    if (tok->isKeyword() && !isCPPCastKeyword(tok) && tok->str().compare(0,8,"operator") != 0)
         return false;
     const Token * parenTok = tok->next();
     if (Token::simpleMatch(parenTok, "<") && parenTok->link())
diff --git a/test/testother.cpp b/test/testother.cpp
@@ -1801,6 +1801,13 @@ class TestOther : public TestFixture {
             check(code, &s64);
             ASSERT_EQUALS("", errout.str());
         }
+
+        check("Writer* getWriter();\n"
+              "\n"
+              "void foo(Buffer& buffer) {\n"
+              "    getWriter()->operator<<(buffer);\n"
+              "}");
+        ASSERT_EQUALS("", errout.str());
     }
 
     void passedByValue_externC() {

Original file line number	Diff line number	Diff line change
`@@ -1801,6 +1801,13 @@ class TestOther : public TestFixture {`
`1801`	`1801`	`check(code, &s64);`
`1802`	`1802`	`ASSERT_EQUALS("", errout.str());`
`1803`	`1803`	`}`
	`1804`	`+`
	`1805`	`+ check("Writer* getWriter();\n"`
	`1806`	`+ "\n"`
	`1807`	`+ "void foo(Buffer& buffer) {\n"`
	`1808`	`+ " getWriter()->operator<<(buffer);\n"`
	`1809`	`+ "}");`
	`1810`	`+ ASSERT_EQUALS("", errout.str());`
`1804`	`1811`	`}`
`1805`	`1812`
`1806`	`1813`	`void passedByValue_externC() {`