You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ws.apache.org by ve...@apache.org on 2012/12/19 19:30:44 UTC
svn commit: r1424010 -
/webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/util/OMSerializerUtil.java
Author: veithen
Date: Wed Dec 19 18:30:44 2012
New Revision: 1424010
URL: http://svn.apache.org/viewvc?rev=1424010&view=rev
Log:
Minor performance improvement: avoid iterator creation when serializing children.
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/util/OMSerializerUtil.java
Modified: webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/util/OMSerializerUtil.java
URL: http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/util/OMSerializerUtil.java?rev=1424010&r1=1424009&r2=1424010&view=diff
==============================================================================
--- webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/util/OMSerializerUtil.java (original)
+++ webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/util/OMSerializerUtil.java Wed Dec 19 18:30:44 2012
@@ -551,9 +551,10 @@ public class OMSerializerUtil {
public static void serializeChildren(OMContainer container, XMLStreamWriter writer,
boolean cache) throws XMLStreamException {
if (cache) {
- Iterator children = container.getChildren();
- while (children.hasNext()) {
- ((OMNodeEx) children.next()).internalSerialize(writer, true);
+ OMNodeEx child = (OMNodeEx)container.getFirstOMChild();
+ while (child != null) {
+ child.internalSerialize(writer, true);
+ child = (OMNodeEx)child.getNextOMSibling();
}
} else {
OMNodeEx child = (OMNodeEx)container.getFirstOMChild();