svn commit: r1080437 - /cxf/trunk/rt/management/src/main/java/org/apache/cxf/management/jmx/InstrumentationManagerImpl.java

[dkulp]

Author: dkulp
Date: Fri Mar 11 04:24:13 2011
New Revision: 1080437

URL: http://svn.apache.org/viewvc?rev=1080437&view=rev
Log:
[CXF-3360] Performance improvement
Patch from Xiaoming Shi applied

Modified:
    
cxf/trunk/rt/management/src/main/java/org/apache/cxf/management/jmx/InstrumentationManagerImpl.java

Modified: 
cxf/trunk/rt/management/src/main/java/org/apache/cxf/management/jmx/InstrumentationManagerImpl.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/rt/management/src/main/java/org/apache/cxf/management/jmx/InstrumentationManagerImpl.java?rev=1080437&r1=1080436&r2=1080437&view=diff
==============================================================================
--- 
cxf/trunk/rt/management/src/main/java/org/apache/cxf/management/jmx/InstrumentationManagerImpl.java
 (original)
+++ 
cxf/trunk/rt/management/src/main/java/org/apache/cxf/management/jmx/InstrumentationManagerImpl.java
 Fri Mar 11 04:24:13 2011
@@ -305,16 +305,23 @@ public class InstrumentationManagerImpl 
     }
     
     private String sanitize(String in) {
-        String result = null;
-        if (in != null) {
-            result = in.replace(':', '_');
-            result = result.replace('/', '_');
-            result = result.replace('\\', '_');
-            result = result.replace('?', '_');
-            result = result.replace('=', '_');
-            result = result.replace(',', '_');
+        StringBuilder str = new StringBuilder(in.length());
+        for (int x = 0; x < str.length(); x++) {
+            char ch = str.charAt(x);
+            switch (ch) {
+            case ':':
+            case '/':
+            case '\\':
+            case '?':
+            case '=':
+            case ',':
+                str.append('_');
+                break;
+            default:
+                str.append(ch);
+            }
         }
-        return result;
+        return str.toString();
     }
 }