zongaro 2002/09/17 14:10:14
Modified: java/src/org/apache/xml/dtm/ref Tag: XSLTC_DTM
DTMDefaultBaseIterators.java
Log:
Removed overriding implementations of getLast in AncestorIterator and
TypedAncestorIterator. They returned index of last node, rather than its
position. Javadoc indicated it was a bug, and caller in XSLTC code expected a
position.
Revision Changes Path
No revision
No revision
1.12.2.5 +0 -36
xml-xalan/java/src/org/apache/xml/dtm/ref/DTMDefaultBaseIterators.java
Index: DTMDefaultBaseIterators.java
===================================================================
RCS file:
/home/cvs/xml-xalan/java/src/org/apache/xml/dtm/ref/DTMDefaultBaseIterators.java,v
retrieving revision 1.12.2.4
retrieving revision 1.12.2.5
diff -u -r1.12.2.4 -r1.12.2.5
--- DTMDefaultBaseIterators.java 29 Jul 2002 00:01:30 -0000 1.12.2.4
+++ DTMDefaultBaseIterators.java 17 Sep 2002 21:10:14 -0000 1.12.2.5
@@ -1480,21 +1480,6 @@
}
/**
- * Returns the last element in this interation.
- *
- * %TBD% %BUG% This is returning a nodeHandle rather than a _position
- * value. That conflicts with what everyone else is doing. And it's
- * talking about the start node, which conflicts with some of the
- * other reverse iterators. DEFINITE BUG; needs to be reconciled.
- *
- * @return the last element in this interation.
- */
- public int getLast()
- {
- return (_startNode);
- }
-
- /**
* Returns a deep copy of this iterator. The cloned iterator is not
reset.
*
* @return a deep copy of this iterator.
@@ -1640,27 +1625,6 @@
}
- /**
- * Returns the last element in this interation.
- *
- * @return the last element in this interation.
- */
- public int getLast()
- {
-
- int last = NULL;
- int curr = _startNode;
-
- while (curr != NULL)
- {
- if (getExpandedTypeID(curr) == _nodeType)
- last = curr;
-
- curr = getParent(curr);
- }
-
- return (last);
- }
} // end of TypedAncestorIterator
/**
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]