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 2014/07/11 22:46:08 UTC

svn commit: r1609830 - in /webservices/axiom/trunk/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom: NodeImpl.java ParentNode.java

Author: veithen
Date: Fri Jul 11 20:46:07 2014
New Revision: 1609830

URL: http://svn.apache.org/r1609830
Log:
Remove unused flag.

Modified:
    webservices/axiom/trunk/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/NodeImpl.java
    webservices/axiom/trunk/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/ParentNode.java

Modified: webservices/axiom/trunk/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/NodeImpl.java
URL: http://svn.apache.org/viewvc/webservices/axiom/trunk/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/NodeImpl.java?rev=1609830&r1=1609829&r2=1609830&view=diff
==============================================================================
--- webservices/axiom/trunk/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/NodeImpl.java (original)
+++ webservices/axiom/trunk/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/NodeImpl.java Fri Jul 11 20:46:07 2014
@@ -72,8 +72,6 @@ public abstract class NodeImpl extends I
      */
     protected final static short HAS_PARENT = 0x1 << 1;
     
-    protected final static short FIRSTCHILD = 0x1 << 2;
-
     /**
      * Used by {@link AttrImpl} to determine whether the attribute has been specified explicitly
      * (flag unset) or has a default value (flag set).
@@ -170,14 +168,6 @@ public abstract class NodeImpl extends I
         flags = (short) (value ? flags | HAS_PARENT : flags & ~HAS_PARENT);
     }
 
-    final boolean isFirstChild() {
-        return (flags & FIRSTCHILD) != 0;
-    }
-
-    final void isFirstChild(boolean value) {
-        flags = (short) (value ? flags | FIRSTCHILD : flags & ~FIRSTCHILD);
-    }
-
     /*
      * DOM-Level 3 methods
      */

Modified: webservices/axiom/trunk/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/ParentNode.java
URL: http://svn.apache.org/viewvc/webservices/axiom/trunk/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/ParentNode.java?rev=1609830&r1=1609829&r2=1609830&view=diff
==============================================================================
--- webservices/axiom/trunk/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/ParentNode.java (original)
+++ webservices/axiom/trunk/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/ParentNode.java Fri Jul 11 20:46:07 2014
@@ -143,7 +143,6 @@ public abstract class ParentNode extends
             if (coreGetLastKnownChild() == null && coreGetFirstChildIfAvailable() == null) {
                 coreSetLastChild((CoreChildNode)newDomChild);
                 coreSetFirstChild((CoreChildNode)newDomChild);
-                ((NodeImpl)coreGetFirstChildIfAvailable()).isFirstChild(true);
                 newDomChild.setParent(this, useDomSemantics);
             } else {
                 ((NodeImpl)coreGetLastKnownChild()).internalSetNextSibling(newDomChild);
@@ -188,8 +187,6 @@ public abstract class ParentNode extends
                             newDomChild.internalSetNextSibling(refDomChild);
                             refDomChild.internalSetPreviousSibling(newDomChild);
 
-                            ((NodeImpl)coreGetFirstChildIfAvailable()).isFirstChild(true);
-                            refDomChild.isFirstChild(false);
                             newDomChild.internalSetPreviousSibling(null); // Just to be
                             // sure :-)