JoergBudi commented on PR #633: URL: https://github.com/apache/commons-collections/pull/633#issuecomment-3181121116
@ppkarwasz : Your suggestion with setNextObject() doesn't work, because you can't "prefetch" the next element without losing the possibility to properly invoke "remove()" down the chain. However with the last commits, i think, there are now no double hasNext() calls anymore. While working on this, I also discovered and fixed a bug with remove() (see the new Testcase IteratorChainTest#testHasNextIsInvokedOnEdgeBeforeRemove, which fails with 4.5.0 . I also followed your suggestion to directly throw an exeption in remove() when the contract is bluntly violated. What do you think ? Any suggestions (e.g. for more testcases or so) ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
