You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@xerces.apache.org by mr...@apache.org on 2009/04/27 06:47:52 UTC

svn commit: r768868 - /xerces/java/branches/xml-schema-1.1-dev/src/org/apache/xerces/dom/NodeImpl.java

Author: mrglavas
Date: Mon Apr 27 04:47:51 2009
New Revision: 768868

URL: http://svn.apache.org/viewvc?rev=768868&view=rev
Log:
Replacing recursion in getElementAncestor() with a loop.

Modified:
    xerces/java/branches/xml-schema-1.1-dev/src/org/apache/xerces/dom/NodeImpl.java

Modified: xerces/java/branches/xml-schema-1.1-dev/src/org/apache/xerces/dom/NodeImpl.java
URL: http://svn.apache.org/viewvc/xerces/java/branches/xml-schema-1.1-dev/src/org/apache/xerces/dom/NodeImpl.java?rev=768868&r1=768867&r2=768868&view=diff
==============================================================================
--- xerces/java/branches/xml-schema-1.1-dev/src/org/apache/xerces/dom/NodeImpl.java (original)
+++ xerces/java/branches/xml-schema-1.1-dev/src/org/apache/xerces/dom/NodeImpl.java Mon Apr 27 04:47:51 2009
@@ -1586,15 +1586,14 @@
         }
     }
 
-
-    Node getElementAncestor (Node currentNode){
+    Node getElementAncestor(Node currentNode) {
         Node parent = currentNode.getParentNode();
-        if (parent != null) {
+        while (parent != null) {
             short type = parent.getNodeType();
             if (type == Node.ELEMENT_NODE) {
                 return parent;
             }
-            return getElementAncestor(parent);
+            parent = parent.getParentNode();
         }
         return null;
     }



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@xerces.apache.org
For additional commands, e-mail: commits-help@xerces.apache.org