From ad0bd0af0962548821a83a9582d96d959f56bae3 Mon Sep 17 00:00:00 2001
From: Alberto Sartori <alberto.sartori@huawei.com>
Date: Thu, 17 Apr 2025 11:19:27 +0200
Subject: ServeFailureReporter: use const& to avoid copies

---
 src/buildtool/build_engine/target_map/absent_target_map.hpp | 2 +-
 src/buildtool/main/main.cpp                                 | 4 +++-
 2 files changed, 4 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/buildtool/build_engine/target_map/absent_target_map.hpp b/src/buildtool/build_engine/target_map/absent_target_map.hpp
index 874204a7..cd06a988 100644
--- a/src/buildtool/build_engine/target_map/absent_target_map.hpp
+++ b/src/buildtool/build_engine/target_map/absent_target_map.hpp
@@ -46,7 +46,7 @@ using AbsentTargetMap = AsyncMapConsumer<ConfiguredTarget, AnalysedTargetPtr>;
 using AbsentTargetVariablesMap =
     AsyncMapConsumer<AbsentTargetDescription, std::vector<std::string>>;
 using ServeFailureLogReporter =
-    std::function<void(ConfiguredTarget, std::string)>;
+    std::function<void(ConfiguredTarget const&, std::string const&)>;
 
 auto CreateAbsentTargetVariablesMap(
     const gsl::not_null<AnalyseContext*>& context,
diff --git a/src/buildtool/main/main.cpp b/src/buildtool/main/main.cpp
index da9aba87..a0aa27fe 100644
--- a/src/buildtool/main/main.cpp
+++ b/src/buildtool/main/main.cpp
@@ -1129,7 +1129,9 @@ auto main(int argc, char* argv[]) -> int {
         auto serve_errors = nlohmann::json::array();
         std::mutex serve_errors_access{};
         BuildMaps::Target::ServeFailureLogReporter collect_serve_errors =
-            [&serve_errors, &serve_errors_access](auto target, auto blob) {
+            [&serve_errors, &serve_errors_access](
+                BuildMaps::Target::ConfiguredTarget const& target,
+                std::string const& blob) {
                 std::unique_lock lock(serve_errors_access);
                 auto target_desc = nlohmann::json::array();
                 target_desc.push_back(target.target.ToJson());
-- 
cgit v1.2.3