You are viewing a plain text version of this content. The canonical link for it is here.
Posted to axis-cvs@ws.apache.org by sa...@apache.org on 2006/03/21 12:57:36 UTC

svn commit: r387507 - /webservices/axis2/trunk/c/modules/xml/om/om_element.c

Author: sahan
Date: Tue Mar 21 03:57:34 2006
New Revision: 387507

URL: http://svn.apache.org/viewcvs?rev=387507&view=rev
Log:
Removed some unnecessary buggy logic in axis2_om_element_get_children_with_qname()

Modified:
    webservices/axis2/trunk/c/modules/xml/om/om_element.c

Modified: webservices/axis2/trunk/c/modules/xml/om/om_element.c
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/c/modules/xml/om/om_element.c?rev=387507&r1=387506&r2=387507&view=diff
==============================================================================
--- webservices/axis2/trunk/c/modules/xml/om/om_element.c (original)
+++ webservices/axis2/trunk/c/modules/xml/om/om_element.c Tue Mar 21 03:57:34 2006
@@ -1057,18 +1057,15 @@
     AXIS2_ENV_CHECK(env, NULL);
     AXIS2_PARAM_CHECK((*env)->error, element_node, NULL);
     om_ele_impl = AXIS2_INTF_TO_IMPL(om_element);
-    if(om_ele_impl->children_qname_iter)
+    if(NULL != om_ele_impl->children_qname_iter)
     {
-        return om_ele_impl->children_qname_iter;
+        AXIS2_OM_CHILDREN_QNAME_ITERATOR_FREE(om_ele_impl->children_qname_iter, env);
+        om_ele_impl->children_qname_iter = NULL;
     }
-    else
-    {
-       om_ele_impl->children_qname_iter =  axis2_om_children_qname_iterator_create(env,
+    om_ele_impl->children_qname_iter =  axis2_om_children_qname_iterator_create(env,
                 AXIS2_OM_NODE_GET_FIRST_CHILD(element_node, env),
                 element_qname);
-        return om_ele_impl->children_qname_iter;                
-    }
-    return NULL;
+    return om_ele_impl->children_qname_iter;                
 }
 
 axis2_om_element_t* AXIS2_CALL