diff --git a/dune/istl/test/solverfactorytest.cc.in b/dune/istl/test/solverfactorytest.cc.in
index 39ecf3cdae0aebd2acba00b6b3bb09ff8efd85ea..c5a519ff7d0a7ac5140069bbe033d1d538a5eae8 100644
--- a/dune/istl/test/solverfactorytest.cc.in
+++ b/dune/istl/test/solverfactorytest.cc.in
@@ -60,7 +60,7 @@ void testSeq(const Dune::ParameterTree& config, Comm c){
     using Operator = Dune::MatrixAdapter<Matrix, Vector, Vector>;
     std::shared_ptr<Operator> op = std::make_shared<Operator>(mat);
 
-    for(std::string test : config.getSubKeys()){
+    for(const std::string& test : config.getSubKeys()){
       Dune::ParameterTree solverConfig = config.sub(test);
       std::cout << " ============== " << test << " ============== " << std::endl;
       try{
@@ -96,7 +96,7 @@ void testOverlapping(const Dune::ParameterTree& config, Comm c){
 
   std::shared_ptr<Operator> op = std::make_shared<Operator>(mat, comm);
 
-  for(std::string test : config.getSubKeys()){
+  for(const std::string& test : config.getSubKeys()){
     Dune::ParameterTree solverConfig = config.sub(test);
     std::cout << " ============== " << test << " ============== " << std::endl;
     try{
@@ -131,7 +131,7 @@ void testNonoverlapping(const Dune::ParameterTree& config, Comm c){
 
   std::shared_ptr<Operator> op = std::make_shared<Operator>(mat, comm);
 
-  for(std::string test : config.getSubKeys()){
+  for(const std::string& test : config.getSubKeys()){
     Dune::ParameterTree solverConfig = config.sub(test);
     std::cout << " ============== " << test << " ============== " << std::endl;
     try{