Michael, keep reminding us of this and I will take a look at it as soon as I have just a bit more time. There is interesting on that class I assure you. (I am interested and I am sure I am not alone.)
Have fun, Paulo > -----Original Message----- > From: James Strachan [mailto:[EMAIL PROTECTED]] > Sent: Friday, February 01, 2002 5:24 PM > To: Jakarta Commons Developers List > Subject: Re: [collections][PATCH] Ambiguous documentation in > FilterIterator > > > Applied - thanks for the patch. > > I don't know the Bag or SequencedHashMap code that well so I'll > hold out for > someone else checking those patches in (and I'm a bit snowed under right > now). > > James > ----- Original Message ----- > From: "Michael Smith" <[EMAIL PROTECTED]> > To: "'Jakarta Commons Developers List'" <[EMAIL PROTECTED]> > Sent: Friday, February 01, 2002 6:07 AM > Subject: [collections][PATCH] Ambiguous documentation in FilterIterator > > > > Ambiguous documentation in > > org/apache/commons/collections/FilterIterator.java > > > > The FilterIterator does not specify whether it *returns* objects that > > match the predicate, or whether it ignores (i.e. does not return) > > objects that match the predicate. This patch clarifies the API > > documentation. > > > > Regards, > > Michael > > > > Index: > > D:/home/michael/dev/jakarta/jakarta-commons/collections/src/java/org/apa > > che/commons/collections/FilterIterator.java > > =================================================================== > > RCS file: > > /home/cvspublic/jakarta-commons/collections/src/java/org/apache/commons/ > > collections/FilterIterator.java,v > > retrieving revision 1.2 > > diff -u -r1.2 FilterIterator.java > > --- > > D:/home/michael/dev/jakarta/jakarta-commons/collections/src/java/org/apa > > che/commons/collections/FilterIterator.java 19 Oct 2001 20:18:21 -0000 > > 1.2 > > +++ > > D:/home/michael/dev/jakarta/jakarta-commons/collections/src/java/org/apa > > che/commons/collections/FilterIterator.java 1 Feb 2002 05:49:32 -0000 > > @@ -11,7 +11,8 @@ > > import java.util.NoSuchElementException; > > > > /** A Proxy {@link Iterator Iterator} which takes a {@link Predicate > > Predicate} instance to filter > > - * out objects from an underlying {@link Iterator Iterator} instance. > > + * out objects from an underlying {@link Iterator Iterator} instance. > > Only objects for which the > > + * specified <code>Predicate</code> evaluates to <code>true</code> are > > returned. > > * > > * @author <a href="mailto:[EMAIL PROTECTED]";>James Strachan</a> > > * @author Jan Sorensen > > > > > ------------------------------------------------------------------ > ---------- > ---- > > > > -- > > To unsubscribe, e-mail: > <mailto:[EMAIL PROTECTED]> > > For additional commands, e-mail: > <mailto:[EMAIL PROTECTED]> > > > _________________________________________________________ > Do You Yahoo!? > Get your free @yahoo.com address at http://mail.yahoo.com > > > -- > To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]> -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
