ConfigBlock GetValueByName: Get rid of temp string for key compare

Load time: 2.03s -> 1.95s
rygorous · Mar 4, 2013 · bbbfb89 · bbbfb89
1 parent b109933
commit bbbfb89
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/SoftwareOcclusionCulling/CPUT/CPUT/CPUTConfigBlock.cpp b/SoftwareOcclusionCulling/CPUT/CPUT/CPUTConfigBlock.cpp
@@ -181,13 +181,18 @@ CPUTConfigEntry *CPUTConfigBlock::AddValue(const cString &szName, const cString
 //----------------------------------------------------------------
 CPUTConfigEntry *CPUTConfigBlock::GetValueByName(const cString &szName)
 {
-    cString szString = szName;
-    std::transform(szString.begin(), szString.end(), szString.begin(), ::tolower);
-
     for(int ii=0; ii<mnValueCount; ++ii)
     {
-        if(mpValues[ii].szName.compare(szString) == 0)
-        {
+		const cString &valName = mpValues[ii].szName;
+		if(valName.size() != szName.size())
+			continue;
+
+		size_t j = 0;
+		while (j < valName.size() && tolower(szName[j]) == valName[j])
+			++j;
+
+		if (j == valName.size()) // match
+		{
             return &mpValues[ii];
         }
     }