From fa282d024e04fb30293409179229628f5e055cc5 Mon Sep 17 00:00:00 2001
From: Klaus Aehlig <klaus.aehlig@huawei.com>
Date: Fri, 13 Sep 2024 12:19:01 +0200
Subject: Avoid unnecessary copies

---
 src/buildtool/build_engine/target_map/target_map.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

(limited to 'src/buildtool/build_engine/target_map/target_map.cpp')

diff --git a/src/buildtool/build_engine/target_map/target_map.cpp b/src/buildtool/build_engine/target_map/target_map.cpp
index 2b9ba3cd..d3fb48b7 100644
--- a/src/buildtool/build_engine/target_map/target_map.cpp
+++ b/src/buildtool/build_engine/target_map/target_map.cpp
@@ -1066,7 +1066,7 @@ void withRuleDefinition(
     std::unordered_map<std::string, ExpressionPtr> params;
     params.reserve(rule->ConfigFields().size() + rule->TargetFields().size() +
                    rule->ImplicitTargetExps().size());
-    for (auto field_name : rule->ConfigFields()) {
+    for (auto const& field_name : rule->ConfigFields()) {
         auto const& field_expression = data->config_exprs[field_name];
         auto field_value = field_expression.Evaluate(
             param_config, {}, [&logger, &field_name](auto const& msg) {
@@ -1197,10 +1197,10 @@ void withRuleDefinition(
 
     std::vector<BuildMaps::Target::ConfiguredTarget> dependency_keys;
     std::vector<BuildMaps::Target::ConfiguredTarget> transition_keys;
-    for (auto target_field_name : rule->TargetFields()) {
+    for (auto const& target_field_name : rule->TargetFields()) {
         auto const& deps_expression = data->target_exprs[target_field_name];
         auto deps_names = deps_expression.Evaluate(
-            param_config, {}, [logger, target_field_name](auto const& msg) {
+            param_config, {}, [&logger, &target_field_name](auto const& msg) {
                 (*logger)(
                     fmt::format("While evaluating target parameter {}:\n{}",
                                 target_field_name,
-- 
cgit v1.2.3