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]