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() ==