From 8a95ed77fec7c2ef3c0fac81258f70db25009b48 Mon Sep 17 00:00:00 2001
From: Thorbjørn Lindeijer <thorbjorn@lindeijer.nl>
Date: Sat, 29 May 2010 19:55:07 +0200
Subject: Use const std::string & to avoid a copy

---
 src/game-server/skillmanager.cpp | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

(limited to 'src/game-server/skillmanager.cpp')

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;
-- 
cgit v1.2.3-70-g09d2