You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by di...@apache.org on 2006/03/20 05:02:08 UTC

svn commit: r387128 - /webservices/axis2/trunk/java/modules/wsdl/src/org/apache/wsdl/impl/WSDLInterfaceImpl.java

Author: dims
Date: Sun Mar 19 20:02:07 2006
New Revision: 387128

URL: http://svn.apache.org/viewcvs?rev=387128&view=rev
Log:
traverse the all hashmap everytime, don't reuse the old iterator even after it has been modified

Modified:
    webservices/axis2/trunk/java/modules/wsdl/src/org/apache/wsdl/impl/WSDLInterfaceImpl.java

Modified: webservices/axis2/trunk/java/modules/wsdl/src/org/apache/wsdl/impl/WSDLInterfaceImpl.java
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/java/modules/wsdl/src/org/apache/wsdl/impl/WSDLInterfaceImpl.java?rev=387128&r1=387127&r2=387128&view=diff
==============================================================================
--- webservices/axis2/trunk/java/modules/wsdl/src/org/apache/wsdl/impl/WSDLInterfaceImpl.java (original)
+++ webservices/axis2/trunk/java/modules/wsdl/src/org/apache/wsdl/impl/WSDLInterfaceImpl.java Sun Mar 19 20:02:07 2006
@@ -77,7 +77,6 @@
             Iterator operationIterator;
             WSDLInterface superInterface;
             WSDLOperation superInterfaceOperation;
-            Iterator thisIterator = all.values().iterator();
             WSDLOperation thisOperation;
             boolean tobeAdded = false;
             while (superIterator.hasNext()) {
@@ -88,6 +87,7 @@
                     superInterfaceOperation =
                             (WSDLOperation) operationIterator.next();
                     tobeAdded = true;
+                    Iterator thisIterator = all.values().iterator();
                     while (thisIterator.hasNext()) {
                         thisOperation = (WSDLOperation) thisIterator.next();
                         if ((thisOperation.getName() ==