valueflow.cpp: avoid unnecessary copy in valueFlowAfterAssign()

firewave · firewave · commit 6983ca2b3800 · 2026-06-23T09:21:09.000+02:00
diff --git a/lib/valueflow.cpp b/lib/valueflow.cpp
@@ -4229,7 +4229,7 @@ static void valueFlowAfterAssign(const TokenList &tokenlist,
                         continue;
                     ids.insert(value.tokvalue->exprId());
                 }
-                for (ValueFlow::Value value : values) {
+                for (ValueFlow::Value& value : values) {
                     if (!value.isSymbolicValue())
                         continue;
                     const Token* expr = value.tokvalue;

Original file line number	Diff line number	Diff line change
`@@ -4229,7 +4229,7 @@ static void valueFlowAfterAssign(const TokenList &tokenlist,`
`4229`	`4229`	`continue;`
`4230`	`4230`	`ids.insert(value.tokvalue->exprId());`
`4231`	`4231`	`}`
`4232`		`- for (ValueFlow::Value value : values) {`
	`4232`	`+ for (ValueFlow::Value& value : values) {`
`4233`	`4233`	`if (!value.isSymbolicValue())`
`4234`	`4234`	`continue;`
`4235`	`4235`	`const Token* expr = value.tokvalue;`