geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From djen...@apache.org
Subject svn commit: r201878 - /geronimo/trunk/modules/axis/src/java/org/apache/geronimo/axis/server/AxisWebServiceContainer.java
Date Sun, 26 Jun 2005 17:38:12 GMT
Author: djencks
Date: Sun Jun 26 10:38:10 2005
New Revision: 201878

URL: http://svn.apache.org/viewcvs?rev=201878&view=rev
Log:
Do not let replaceAll lull me into forgetting that strings are immutable

Modified:
    geronimo/trunk/modules/axis/src/java/org/apache/geronimo/axis/server/AxisWebServiceContainer.java

Modified: geronimo/trunk/modules/axis/src/java/org/apache/geronimo/axis/server/AxisWebServiceContainer.java
URL: http://svn.apache.org/viewcvs/geronimo/trunk/modules/axis/src/java/org/apache/geronimo/axis/server/AxisWebServiceContainer.java?rev=201878&r1=201877&r2=201878&view=diff
==============================================================================
--- geronimo/trunk/modules/axis/src/java/org/apache/geronimo/axis/server/AxisWebServiceContainer.java
(original)
+++ geronimo/trunk/modules/axis/src/java/org/apache/geronimo/axis/server/AxisWebServiceContainer.java
Sun Jun 26 10:38:10 2005
@@ -253,8 +253,8 @@
                     realLocation.getPath(), //TODO this is very wrong for services containing
more than one port.
                     null,
                     null);
-            ((String)wsdl).replaceAll(LOCATION_REPLACEMENT_TOKEN, updated.toString());
-            response.getOutputStream().write(((String)wsdl).getBytes());
+            String replaced = ((String)wsdl).replaceAll(LOCATION_REPLACEMENT_TOKEN, updated.toString());
+            response.getOutputStream().write(replaced.getBytes());
 //        } else {
 //             Definition definition = (Definition) wsdl;
 //            synchronized (definition) {



Mime
View raw message