Only reset reference at function declarator parentheses

`int& (r)` and `int& UNUSED(r)` lost their reference flag, which gave
uninitvar false positives.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Author: autoantwort

lib/symboldatabase.cpp

@@ -2595,7 +2595,7 @@ void Variable::evaluate(const Settings& settings)
         } else if (tok->str() == "&&") { // Before simplification, && isn't split up
             setFlag(fIsRValueRef, true);
             setFlag(fIsReference, true); // Set also fIsReference
-        } else if (tok->str() == "(") {
+        } else if (tok->str() == "(" && Token::simpleMatch(tok->link(), ") (")) {
             // a reference before the parentheses belongs to the return type of a function pointer: int& (*f)()
             setFlag(fIsRValueRef, false);
             setFlag(fIsReference, false);

test/testsymboldatabase.cpp

@@ -1637,6 +1637,18 @@ class TestSymbolDatabase : public TestFixture {
             ASSERT(p->valueType());
             ASSERT(p->valueType()->reference == Reference::None);
         }
+        { // parentheses that are no function declarator
+            GET_SYMBOL_DB("void foo(int& (r)) {}\n");
+            const Variable* const p = db->functionScopes.front()->function->getArgumentVar(0);
+            ASSERT(p);
+            ASSERT(p->isReference());
+        }
+        {
+            GET_SYMBOL_DB("void foo(int& UNUSED(r)) {}\n");
+            const Variable* const p = db->functionScopes.front()->function->getArgumentVar(0);
+            ASSERT(p);
+            ASSERT(p->isReference());
+        }
         { // reference to function pointer
             GET_SYMBOL_DB("void foo(int (*&f)()) {}\n");
             const Variable* const p = db->getVariableFromVarId(1);