stefanvodita commented on PR #12287: URL: https://github.com/apache/lucene/pull/12287#issuecomment-1646142364
I should have been clearer. I'm assuming the existing tests that call `nextPolygon()` are passing before and after the fix, which warrants some suspicion. Could we devise a test that would use these utility methods and fail without this fix? It sounds like you were already writing a test that would serve this purpose. Let me know if I've got it wrong. -- 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: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
