valueflow.cpp: avoid unnecessary copies in valueFlowContainerSize()

Author: firewave

lib/valueflow.cpp

@@ -6657,8 +6657,8 @@ static void valueFlowContainerSize(const TokenList& tokenlist,
             continue;
         }
 
-        for (const ValueFlow::Value& value : values) {
-            valueFlowForward(nameToken->next(), var->nameToken(), value, tokenlist, errorLogger, settings);
+        for (ValueFlow::Value& value : values) {
+            valueFlowForward(nameToken->next(), var->nameToken(), std::move(value), tokenlist, errorLogger, settings);
         }
     }
 
@@ -6704,8 +6704,8 @@ static void valueFlowContainerSize(const TokenList& tokenlist,
                     const Token* constructorArgs = tok;
                     values = getContainerSizeFromConstructor(constructorArgs, tok->valueType(), settings, true);
                 }
-                for (const ValueFlow::Value& value : values)
-                    setTokenValue(tok, value, settings);
+                for (ValueFlow::Value& value : values)
+                    setTokenValue(tok, std::move(value), settings);
             }
             else if (Token::Match(tok->previous(), ",|( {|%str%")) {
                 int nArg{};