From ce85f329c34deafe228cb0b12a39095227631701 Mon Sep 17 00:00:00 2001
From: Simon Dold <simon.dold@stud.unibas.ch>
Date: Fri, 9 Feb 2024 17:42:02 +0100
Subject: [PATCH] [issue1082] reduce tox complaints.

---
 src/search/search_algorithms/eager_search.cc           | 10 +++++-----
 src/search/search_algorithms/eager_search.h            | 10 +++++-----
 .../search_algorithms/enforced_hill_climbing_search.cc |  4 ++--
 .../search_algorithms/enforced_hill_climbing_search.h  |  4 ++--
 src/search/search_algorithms/lazy_search.cc            |  4 ++--
 src/search/search_algorithms/lazy_search.h             |  4 ++--
 6 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/src/search/search_algorithms/eager_search.cc b/src/search/search_algorithms/eager_search.cc
index 0715fd8919..a15c90bd2a 100644
--- a/src/search/search_algorithms/eager_search.cc
+++ b/src/search/search_algorithms/eager_search.cc
@@ -20,12 +20,12 @@ using namespace std;
 
 namespace eager_search {
 EagerSearch::EagerSearch(
-    shared_ptr<OpenListFactory> open,
+    const shared_ptr<OpenListFactory> &open,
     bool reopen_closed,
-    shared_ptr<Evaluator> f_eval,
-    vector<shared_ptr<Evaluator>> preferred,
-    shared_ptr<PruningMethod> pruning,
-    shared_ptr<Evaluator> lazy_evaluator,
+    const shared_ptr<Evaluator> &f_eval,
+    const vector<shared_ptr<Evaluator>> &preferred,
+    const shared_ptr<PruningMethod> &pruning,
+    const shared_ptr<Evaluator> &lazy_evaluator,
     OperatorCost cost_type,
     int bound,
     double max_time,
diff --git a/src/search/search_algorithms/eager_search.h b/src/search/search_algorithms/eager_search.h
index 0057e5292c..7ddc5efbcd 100644
--- a/src/search/search_algorithms/eager_search.h
+++ b/src/search/search_algorithms/eager_search.h
@@ -38,12 +38,12 @@ class EagerSearch : public SearchAlgorithm {
 
 public:
     explicit EagerSearch(
-        std::shared_ptr<OpenListFactory> open,
+        const std::shared_ptr<OpenListFactory> &open,
         bool reopen_closed,
-        std::shared_ptr<Evaluator> f_eval,
-        std::vector<std::shared_ptr<Evaluator>> preferred,
-        std::shared_ptr<PruningMethod> pruning,
-        std::shared_ptr<Evaluator> lazy_evaluator,
+        const std::shared_ptr<Evaluator> &f_eval,
+        const std::vector<std::shared_ptr<Evaluator>> &preferred,
+        const std::shared_ptr<PruningMethod> &pruning,
+        const std::shared_ptr<Evaluator> &lazy_evaluator,
         OperatorCost cost_type,
         int bound,
         double max_time,
diff --git a/src/search/search_algorithms/enforced_hill_climbing_search.cc b/src/search/search_algorithms/enforced_hill_climbing_search.cc
index c4119e3283..f624cf5bd7 100644
--- a/src/search/search_algorithms/enforced_hill_climbing_search.cc
+++ b/src/search/search_algorithms/enforced_hill_climbing_search.cc
@@ -66,9 +66,9 @@ static shared_ptr<OpenListFactory> create_ehc_open_list_factory(
 
 
 EnforcedHillClimbingSearch::EnforcedHillClimbingSearch(
-    shared_ptr<Evaluator> h,
+    const shared_ptr<Evaluator> &h,
     PreferredUsage preferred_usage,
-    vector<shared_ptr<Evaluator>> preferred,
+    const vector<shared_ptr<Evaluator>> &preferred,
     OperatorCost cost_type,
     int bound,
     double max_time,
diff --git a/src/search/search_algorithms/enforced_hill_climbing_search.h b/src/search/search_algorithms/enforced_hill_climbing_search.h
index 643b056223..fb1c7ca2a4 100644
--- a/src/search/search_algorithms/enforced_hill_climbing_search.h
+++ b/src/search/search_algorithms/enforced_hill_climbing_search.h
@@ -61,9 +61,9 @@ class EnforcedHillClimbingSearch : public SearchAlgorithm {
 
 public:
     EnforcedHillClimbingSearch(
-        std::shared_ptr<Evaluator> h,
+        const std::shared_ptr<Evaluator> &h,
         PreferredUsage preferred_usage,
-        std::vector<std::shared_ptr<Evaluator>> preferred,
+        const std::vector<std::shared_ptr<Evaluator>> &preferred,
         OperatorCost cost_type,
         int bound,
         double max_time,
diff --git a/src/search/search_algorithms/lazy_search.cc b/src/search/search_algorithms/lazy_search.cc
index 54ef63235d..e0525a3c0e 100644
--- a/src/search/search_algorithms/lazy_search.cc
+++ b/src/search/search_algorithms/lazy_search.cc
@@ -18,9 +18,9 @@ using namespace std;
 
 namespace lazy_search {
 LazySearch::LazySearch(
-    shared_ptr<OpenListFactory> open,
+    const shared_ptr<OpenListFactory> &open,
     bool reopen_closed,
-    vector<shared_ptr<Evaluator>> &preferred,
+    const vector<shared_ptr<Evaluator>> &preferred,
     bool randomize_successors,
     bool preferred_successors_first,
     int random_seed,
diff --git a/src/search/search_algorithms/lazy_search.h b/src/search/search_algorithms/lazy_search.h
index ed258d11ff..5977a5631b 100644
--- a/src/search/search_algorithms/lazy_search.h
+++ b/src/search/search_algorithms/lazy_search.h
@@ -50,9 +50,9 @@ class LazySearch : public SearchAlgorithm {
 
 public:
     LazySearch(
-        std::shared_ptr<OpenListFactory> open,
+        const std::shared_ptr<OpenListFactory> &open,
         bool reopen_closed,
-        std::vector<std::shared_ptr<Evaluator>> &evaluators,
+        const std::vector<std::shared_ptr<Evaluator>> &evaluators,
         bool randomize_successors,
         bool preferred_successors_first,
         int random_seed,