[CL] Avoid string copies. Use move

ChemistryLib/PhreeqcKernelData/CreateKineticReactant.cpp

@@ -49,7 +49,7 @@ std::unique_ptr<Kinetics> createKineticReactants(
             1);
         std::fill(std::begin(*amount), std::end(*amount), initial_amount);
 
-        kinetic_reactants.emplace_back(name, initial_amount);
+        kinetic_reactants.emplace_back(std::move(name), initial_amount);
     }
 
     return std::make_unique<Kinetics>(kinetic_reactants);

ChemistryLib/PhreeqcKernelData/KineticReactant.cpp

@@ -16,11 +16,10 @@ namespace ChemistryLib
 {
 namespace PhreeqcKernelData
 {
-KineticReactant::KineticReactant(std::string const& name,
-                                 double const initial_amount)
+KineticReactant::KineticReactant(std::string name, double const initial_amount)
 {
-    rate_name = name.c_str();
-    namecoef.add(name.c_str(), 1.0);
+    rate_name = std::move(name);
+    namecoef.add(rate_name.c_str(), 1.0);
     m = initial_amount;
     m0 = initial_amount;
 }

ChemistryLib/PhreeqcKernelData/KineticReactant.h

@@ -22,7 +22,7 @@ namespace PhreeqcKernelData
 class KineticReactant final : private cxxKineticsComp
 {
 public:
-    KineticReactant(std::string const& name, double const initial_amount);
+    KineticReactant(std::string name, double const initial_amount);
 
     cxxKineticsComp const* castToBaseClass() const
     {