[issue1082] reduce tox complaints.

aibasel · Feb 9, 2024 · ce85f32 · ce85f32
1 parent d73e301
commit ce85f32
Show file tree

Hide file tree

Showing 6 changed files with 18 additions and 18 deletions.
diff --git 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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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,