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 2016/03/16 23:44:03 UTC
svn commit: r1735334 -
/webservices/axiom/trunk/aspects/om-aspects/src/main/java/org/apache/axiom/om/impl/mixin/AxiomContainerSupport.aj
Author: veithen
Date: Wed Mar 16 22:44:03 2016
New Revision: 1735334
URL: http://svn.apache.org/viewvc?rev=1735334&view=rev
Log:
Rewrite the getChildrenWithName implementation.
Modified:
webservices/axiom/trunk/aspects/om-aspects/src/main/java/org/apache/axiom/om/impl/mixin/AxiomContainerSupport.aj
Modified: webservices/axiom/trunk/aspects/om-aspects/src/main/java/org/apache/axiom/om/impl/mixin/AxiomContainerSupport.aj
URL: http://svn.apache.org/viewvc/webservices/axiom/trunk/aspects/om-aspects/src/main/java/org/apache/axiom/om/impl/mixin/AxiomContainerSupport.aj?rev=1735334&r1=1735333&r2=1735334&view=diff
==============================================================================
--- webservices/axiom/trunk/aspects/om-aspects/src/main/java/org/apache/axiom/om/impl/mixin/AxiomContainerSupport.aj (original)
+++ webservices/axiom/trunk/aspects/om-aspects/src/main/java/org/apache/axiom/om/impl/mixin/AxiomContainerSupport.aj Wed Mar 16 22:44:03 2016
@@ -178,9 +178,10 @@ public aspect AxiomContainerSupport {
Mappers.<OMElement>identity(), AxiomSemantics.INSTANCE);
}
- // TODO: DOOM actually supported elementQName == null; need to test and document this
- public Iterator<OMElement> AxiomContainer.getChildrenWithName(QName elementQName) {
- return new OMChildrenQNameIterator(getFirstOMChild(), elementQName);
+ public Iterator<OMElement> AxiomContainer.getChildrenWithName(QName name) {
+ return coreGetElements(Axis.CHILDREN, AxiomElement.class,
+ ElementMatcher.BY_QNAME, name.getNamespaceURI(), name.getLocalPart(),
+ Mappers.<OMElement>identity(), AxiomSemantics.INSTANCE);
}
public Iterator<OMSerializable> AxiomContainer.getDescendants(boolean includeSelf) {