valueflow.cpp: avoid unnecessary copies in valueFlowInferCondition()

firewave · firewave · commit 5017e3e77f32 · 2026-06-23T09:21:09.000+02:00
diff --git a/lib/valueflow.cpp b/lib/valueflow.cpp
@@ -5078,7 +5078,7 @@ static void valueFlowInferCondition(TokenList& tokenlist, const Settings& settin
                 for (const ValuePtr<InferModel>& model : iteratorModels) {
                     std::vector<ValueFlow::Value> result =
                         infer(model, tok->str(), tok->astOperand1()->values(), tok->astOperand2()->values());
-                    for (ValueFlow::Value value : result) {
+                    for (ValueFlow::Value& value : result) {
                         value.valueType = ValueFlow::Value::ValueType::INT;
                         setTokenValue(tok, std::move(value), settings);
                     }

Original file line number	Diff line number	Diff line change
`@@ -5078,7 +5078,7 @@ static void valueFlowInferCondition(TokenList& tokenlist, const Settings& settin`
`5078`	`5078`	`for (const ValuePtr<InferModel>& model : iteratorModels) {`
`5079`	`5079`	`std::vector<ValueFlow::Value> result =`
`5080`	`5080`	`infer(model, tok->str(), tok->astOperand1()->values(), tok->astOperand2()->values());`
`5081`		`- for (ValueFlow::Value value : result) {`
	`5081`	`+ for (ValueFlow::Value& value : result) {`
`5082`	`5082`	`value.valueType = ValueFlow::Value::ValueType::INT;`
`5083`	`5083`	`setTokenValue(tok, std::move(value), settings);`
`5084`	`5084`	`}`