SC-10749 commented on PR #335: URL: https://github.com/apache/commons-math/pull/335#issuecomment-6335804426
> Thanks for the PR. The tests are valid. The code change should apply the fix to the `Incrementor` class. This change drops use of the `Incrementor` and creates a new `Iterator` implementation. Downstream classes that use the `Incrementor` will not benefit from the change. Thanks for your review and feedback! I understand your point. I will revise the fix directly inside the Incrementor class instead of introducing a new Iterator implementation, so that all downstream classes relying on Incrementor can get this fix. I'll push the updated changes soon. -- 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]
