diff options
author | Michi Henning <michi@zeroc.com> | 2009-05-14 10:59:11 +1000 |
---|---|---|
committer | Michi Henning <michi@zeroc.com> | 2009-05-14 10:59:11 +1000 |
commit | 8618940aad4d84f336d3314ea3b8c5ebe5cba2a5 (patch) | |
tree | e6c918f1061ffcc6c1ac826289b8fc4cbdbebd1e /java/src/IceInternal/ReferenceFactory.java | |
parent | Bug 3995: slicing/object test fails (diff) | |
download | ice-8618940aad4d84f336d3314ea3b8c5ebe5cba2a5.tar.bz2 ice-8618940aad4d84f336d3314ea3b8c5ebe5cba2a5.tar.xz ice-8618940aad4d84f336d3314ea3b8c5ebe5cba2a5.zip |
Fixed performance issues reported by FindBugs. Made corresponding
changes for C# where applicable.
Diffstat (limited to 'java/src/IceInternal/ReferenceFactory.java')
-rw-r--r-- | java/src/IceInternal/ReferenceFactory.java | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/java/src/IceInternal/ReferenceFactory.java b/java/src/IceInternal/ReferenceFactory.java index 4eaebed5bfc..d329b6249a0 100644 --- a/java/src/IceInternal/ReferenceFactory.java +++ b/java/src/IceInternal/ReferenceFactory.java @@ -418,12 +418,14 @@ public final class ReferenceFactory else if(unknownEndpoints.size() != 0 && _instance.initializationData().properties.getPropertyAsIntWithDefault("Ice.Warn.Endpoints", 1) > 0) { - String msg = "Proxy contains unknown endpoints:"; + StringBuffer msg = new StringBuffer("Proxy contains unknown endpoints:"); for(String e : unknownEndpoints) { - msg += " `" + e + "'"; + msg.append(" `"); + msg.append(e); + msg.append("'"); } - _instance.initializationData().logger.warning(msg); + _instance.initializationData().logger.warning(msg.toString()); } EndpointI[] endp = new EndpointI[endpoints.size()]; @@ -687,12 +689,15 @@ public final class ReferenceFactory if(unknownProps.size() != 0) { - String message = "found unknown properties for proxy '" + prefix + "':"; + StringBuffer message = new StringBuffer("found unknown properties for proxy '"); + message.append(prefix); + message.append("':"); for(String s : unknownProps) { - message += "\n " + s; + message.append("\n "); + message.append(s); } - _instance.initializationData().logger.warning(message); + _instance.initializationData().logger.warning(message.toString()); } } |