Remove more warning in simulation control

chaos-polymtl · Aug 16, 2024 · 1ed0261 · 1ed0261
1 parent e3ed1c0
commit 1ed0261
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 12 deletions.
diff --git a/include/core/simulation_control.h b/include/core/simulation_control.h
@@ -165,7 +165,7 @@ class SimulationControl
    *
    **/
 
-  SimulationControl(const Parameters::SimulationControl param);
+  SimulationControl(const Parameters::SimulationControl &param);
 
   /**
    * @brief Pure virtual function to control the progression of the simulation.
@@ -526,7 +526,7 @@ class SimulationControlTransient : public SimulationControl
 
 
 public:
-  SimulationControlTransient(Parameters::SimulationControl param);
+  SimulationControlTransient(const Parameters::SimulationControl &param);
 
   virtual void
   print_progression(const ConditionalOStream &pcout) override;
@@ -551,7 +551,7 @@ class SimulationControlTransient : public SimulationControl
 class SimulationControlTransientDEM : public SimulationControlTransient
 {
 public:
-  SimulationControlTransientDEM(Parameters::SimulationControl param);
+  SimulationControlTransientDEM(const Parameters::SimulationControl &param);
 
   virtual void
   print_progression(const ConditionalOStream &pcout) override;
@@ -579,7 +579,8 @@ class SimulationControlTransientDynamicOutput
   calculate_time_step() override;
 
 public:
-  SimulationControlTransientDynamicOutput(Parameters::SimulationControl param);
+  SimulationControlTransientDynamicOutput(
+    const Parameters::SimulationControl &param);
 
 
   /**
@@ -593,7 +594,7 @@ class SimulationControlTransientDynamicOutput
 class SimulationControlSteady : public SimulationControl
 {
 public:
-  SimulationControlSteady(Parameters::SimulationControl param);
+  SimulationControlSteady(const Parameters::SimulationControl &param);
 
   virtual void
   print_progression(const ConditionalOStream &pcout) override;
@@ -614,7 +615,7 @@ class SimulationControlSteady : public SimulationControl
 class SimulationControlAdjointSteady : public SimulationControlTransient
 {
 public:
-  SimulationControlAdjointSteady(Parameters::SimulationControl param);
+  SimulationControlAdjointSteady(const Parameters::SimulationControl &param);
 
   virtual void
   print_progression(const ConditionalOStream &pcout) override;

diff --git a/source/core/simulation_control.cc b/source/core/simulation_control.cc
@@ -6,7 +6,7 @@
 #include <sstream>
 
 
-SimulationControl::SimulationControl(const Parameters::SimulationControl param)
+SimulationControl::SimulationControl(const Parameters::SimulationControl &param)
   : method(param.method)
   , assembly_method(param.method)
   , current_time(0)
@@ -211,7 +211,7 @@ SimulationControl::get_checkpointed_simulation_control_info(
 }
 
 SimulationControlTransient::SimulationControlTransient(
-  Parameters::SimulationControl param)
+  const Parameters::SimulationControl &param)
   : SimulationControl(param)
   , adapt(param.adapt)
   , adaptative_time_step_scaling(param.adaptative_time_step_scaling)
@@ -288,7 +288,7 @@ SimulationControlTransient::calculate_time_step()
 }
 
 SimulationControlTransientDEM::SimulationControlTransientDEM(
-  Parameters::SimulationControl param)
+  const Parameters::SimulationControl &param)
   : SimulationControlTransient(param)
 {}
 
@@ -313,7 +313,8 @@ SimulationControlTransientDEM::print_progression(
 
 
 SimulationControlTransientDynamicOutput::
-  SimulationControlTransientDynamicOutput(Parameters::SimulationControl param)
+  SimulationControlTransientDynamicOutput(
+    const Parameters::SimulationControl &param)
   : SimulationControlTransient(param)
   , time_step_forced_output(false)
   // To be fixed for restarts
@@ -368,7 +369,7 @@ SimulationControlTransientDynamicOutput::is_output_iteration()
 
 
 SimulationControlSteady::SimulationControlSteady(
-  Parameters::SimulationControl param)
+  const Parameters::SimulationControl &param)
   : SimulationControl(param)
 {}
 
@@ -439,7 +440,7 @@ SimulationControlAdjointSteady::is_at_end()
 }
 
 SimulationControlAdjointSteady::SimulationControlAdjointSteady(
-  Parameters::SimulationControl param)
+  const Parameters::SimulationControl &param)
   : SimulationControlTransient(param)
 {}