Pass actions and trees in analysis result as shared pointer

... to avoid unnecessary copying and moving of larger objects.

1 files changed, 3 insertions, 3 deletions

diff --git a/src/buildtool/build_engine/target_map/utils.hpp b/src/buildtool/build_engine/target_map/utils.hpp
index e92e6281..f805cde3 100644
--- a/src/buildtool/build_engine/target_map/utils.hpp
+++ b/src/buildtool/build_engine/target_map/utils.hpp
@@ -43,13 +43,13 @@ auto getTainted(std::set<std::string>* tainted,
                 const ExpressionPtr& tainted_exp,
                 const BuildMaps::Target::TargetMap::LoggerPtr& logger) -> bool;
 
-auto createAction(ActionDescription::outputs_t output_files,
-                  ActionDescription::outputs_t output_dirs,
+auto createAction(const ActionDescription::outputs_t& output_files,
+                  const ActionDescription::outputs_t& output_dirs,
                   std::vector<std::string> command,
                   const ExpressionPtr& env,
                   std::optional<std::string> may_fail,
                   bool no_cache,
-                  const ExpressionPtr& inputs_exp) -> ActionDescription;
+                  const ExpressionPtr& inputs_exp) -> ActionDescription::Ptr;
 
 }  // namespace BuildMaps::Target::Utils
 #endif