You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by co...@apache.org on 2012/10/25 12:51:25 UTC

svn commit: r1402094 - /cxf/trunk/rt/core/src/main/java/org/apache/cxf/wsdl11/WSDLManagerImpl.java

Author: coheigea
Date: Thu Oct 25 10:51:25 2012
New Revision: 1402094

URL: http://svn.apache.org/viewvc?rev=1402094&view=rev
Log:
Use url.toString() instead of storing the URL directly in definitionsMap

Modified:
    cxf/trunk/rt/core/src/main/java/org/apache/cxf/wsdl11/WSDLManagerImpl.java

Modified: cxf/trunk/rt/core/src/main/java/org/apache/cxf/wsdl11/WSDLManagerImpl.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/core/src/main/java/org/apache/cxf/wsdl11/WSDLManagerImpl.java?rev=1402094&r1=1402093&r2=1402094&view=diff
==============================================================================
--- cxf/trunk/rt/core/src/main/java/org/apache/cxf/wsdl11/WSDLManagerImpl.java (original)
+++ cxf/trunk/rt/core/src/main/java/org/apache/cxf/wsdl11/WSDLManagerImpl.java Thu Oct 25 10:51:25 2012
@@ -166,14 +166,15 @@ public class WSDLManagerImpl implements 
      * @see org.apache.cxf.wsdl.WSDLManager#getDefinition(java.net.URL)
      */
     public Definition getDefinition(URL url) throws WSDLException { 
+        String urlString = url.toString();
         synchronized (definitionsMap) {
-            if (definitionsMap.containsKey(url)) {
-                return definitionsMap.get(url);
+            if (definitionsMap.containsKey(urlString)) {
+                return definitionsMap.get(urlString);
             }
         }
-        Definition def = loadDefinition(url.toString());
+        Definition def = loadDefinition(urlString);
         synchronized (definitionsMap) {
-            definitionsMap.put(url, def);
+            definitionsMap.put(urlString, def);
         }
         return def;
     }