Unless I'm dreaming, the return was removed not added. Stephen ----- Original Message ----- From: "pwomack" <[EMAIL PROTECTED]> > Stephen Colebourne wrote: > > I have a vague memory of something like this getting fixed once upon a time. > > Are you testing against the CVS version of this file or a release? > > Oh darn. > > It looks like there's a fundemental "nasty", which may require > thought to fix. > > *my* fault is caused in updateCurrentIterator(), > which returns IMMEDIATELY after setting > currentIterator = (Iterator) iteratorChain.get(0); > > If this currentIterator is empty, IteratorChain.hasNext() will return > false, and it's all over. > > But this "return" in updateCurrentIterator() was added as a result > (between v1.2 and 1.3 > http://cvs.apache.org/viewcvs/jakarta-commons/collections/src/java/org/apach e/commons/collections/iterators/IteratorChain.java.diff?r1=1.2&r2=1.3&diff_f ormat=h > as result of a bug report: > > "fix bug pointed out bu Jonathan Carlson > add test that demonstrates" > > So I guess fixing one bug has caused another. > > BugBear > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] >
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
