Use const std::string & to avoid a copy

1 files changed, 4 insertions, 5 deletions

diff --git a/src/game-server/skillmanager.cpp b/src/game-server/skillmanager.cpp
index ccf15923..4d262a4a 100644
--- a/src/game-server/skillmanager.cpp
+++ b/src/game-server/skillmanager.cpp
@@ -102,17 +102,16 @@ void SkillManager::reload()
     }
 }
 
-int SkillManager::getIdFromString(std::string name)
+int SkillManager::getIdFromString(const std::string &name)
 {
     //check if already an integer, if yes just return it
     int val;
     val = atoi(name.c_str());
-    if (val) return val;
+    if (val)
+        return val;
 
     // convert to upper case for easier finding
-    name = utils::toupper(name);
-    // find it
-    SkillMap::iterator i = skillMap.find(name);
+    SkillMap::iterator i = skillMap.find(utils::toupper(name));
     if (i == skillMap.end())
     {
         return 0;