Update clang-tidy to clang version 8

[alexxy/gromacs.git] / src / gromacs / mdrun / runner.cpp

diff --git a/src/gromacs/mdrun/runner.cpp b/src/gromacs/mdrun/runner.cpp
index bc1a4bbc6ee418f80ba64e57faccf667526ed0f8..4a7622c19b8cb1da7e41e3e2e3a8daccd64eaddb 100644 (file)
--- a/src/gromacs/mdrun/runner.cpp
+++ b/src/gromacs/mdrun/runner.cpp
@@ -1635,7 +1635,7 @@ Mdrunner::~Mdrunner()
 };
 
 void Mdrunner::addPotential(std::shared_ptr<gmx::IRestraintPotential> puller,
-                            std::string                               name)
+                            const std::string                        &name)
 {
     GMX_ASSERT(restraintManager_, "Mdrunner must have a restraint manager.");
     // Not sure if this should be logged through the md logger or something else,
@@ -1645,7 +1645,7 @@ void Mdrunner::addPotential(std::shared_ptr<gmx::IRestraintPotential> puller,
     // When multiple restraints are used, it may be wasteful to register them separately.
     // Maybe instead register an entire Restraint Manager as a force provider.
     restraintManager_->addToSpec(std::move(puller),
-                                 std::move(name));
+                                 name);
 }
 
 Mdrunner::Mdrunner(std::unique_ptr<MDModules> mdModules)