by reference

CarloVarni · Oct 30, 2024 · dfeefe4 · dfeefe4
1 parent b9216d2
commit dfeefe4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/Core/include/Acts/Seeding/SeedFilter.hpp b/Core/include/Acts/Seeding/SeedFilter.hpp
@@ -38,9 +38,9 @@ struct SeedFilterState {
 template <typename external_spacepoint_t>
 class SeedFilter final {
  public:
-  SeedFilter(SeedFilterConfig config,
+  SeedFilter(const SeedFilterConfig& config,
              IExperimentCuts<external_spacepoint_t>* expCuts = nullptr);
-  SeedFilter(SeedFilterConfig config,
+  SeedFilter(const SeedFilterConfig& config,
              std::unique_ptr<const Acts::Logger> logger,
              IExperimentCuts<external_spacepoint_t>* expCuts = nullptr);
   SeedFilter(const SeedFilter<external_spacepoint_t>&) = delete;

diff --git a/Core/include/Acts/Seeding/SeedFilter.ipp b/Core/include/Acts/Seeding/SeedFilter.ipp
@@ -16,9 +16,9 @@ namespace Acts {
 // constructor
 template <typename external_spacepoint_t>
 SeedFilter<external_spacepoint_t>::SeedFilter(
-    SeedFilterConfig config,
+    const SeedFilterConfig& config,
     IExperimentCuts<external_spacepoint_t>* expCuts /* = 0*/)
-    : m_cfg(std::move(config)), m_experimentCuts(expCuts) {
+    : m_cfg(config), m_experimentCuts(expCuts) {
   if (!config.isInInternalUnits) {
     throw std::runtime_error(
         "SeedFilterConfig not in ACTS internal units in SeedFilter");
@@ -27,11 +27,9 @@ SeedFilter<external_spacepoint_t>::SeedFilter(
 
 template <typename external_spacepoint_t>
 SeedFilter<external_spacepoint_t>::SeedFilter(
-    SeedFilterConfig config, std::unique_ptr<const Acts::Logger> logger,
+    const SeedFilterConfig& config, std::unique_ptr<const Acts::Logger> logger,
     IExperimentCuts<external_spacepoint_t>* expCuts /* = 0*/)
-    : m_cfg(std::move(config)),
-      m_logger(std::move(logger)),
-      m_experimentCuts(expCuts) {
+    : m_cfg(config), m_logger(std::move(logger)), m_experimentCuts(expCuts) {
   if (!config.isInInternalUnits) {
     throw std::runtime_error(
         "SeedFilterConfig not in ACTS internal units in SeedFilter");