You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by dk...@apache.org on 2011/03/11 05:24:14 UTC
svn commit: r1080437 -
/cxf/trunk/rt/management/src/main/java/org/apache/cxf/management/jmx/InstrumentationManagerImpl.java
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();
}
}