You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-commits@axis.apache.org by ve...@apache.org on 2011/01/02 20:43:51 UTC
svn commit: r1054463 -
/axis/axis2/java/core/trunk/modules/addressing/src/org/apache/axis2/handlers/addressing/AddressingOutHandler.java
Author: veithen
Date: Sun Jan 2 19:43:51 2011
New Revision: 1054463
URL: http://svn.apache.org/viewvc?rev=1054463&view=rev
Log:
Avoid invalid use of iterator: removing items while iterating over a collection should be done with Iterator#remove().
Modified:
axis/axis2/java/core/trunk/modules/addressing/src/org/apache/axis2/handlers/addressing/AddressingOutHandler.java
Modified: axis/axis2/java/core/trunk/modules/addressing/src/org/apache/axis2/handlers/addressing/AddressingOutHandler.java
URL: http://svn.apache.org/viewvc/axis/axis2/java/core/trunk/modules/addressing/src/org/apache/axis2/handlers/addressing/AddressingOutHandler.java?rev=1054463&r1=1054462&r2=1054463&view=diff
==============================================================================
--- axis/axis2/java/core/trunk/modules/addressing/src/org/apache/axis2/handlers/addressing/AddressingOutHandler.java (original)
+++ axis/axis2/java/core/trunk/modules/addressing/src/org/apache/axis2/handlers/addressing/AddressingOutHandler.java Sun Jan 2 19:43:51 2011
@@ -565,8 +565,8 @@ public class AddressingOutHandler extend
QName qname = new QName(addressingNamespace, name, WSA_DEFAULT_PREFIX);
Iterator iterator = header.getChildrenWithName(qname);
while (iterator.hasNext()) {
- OMElement addressingHeader = (OMElement)iterator.next();
- addressingHeader.detach();
+ iterator.next();
+ iterator.remove();
}
}
} else {