paulk-asert commented on PR #1915: URL: https://github.com/apache/groovy/pull/1915#issuecomment-1657355755
@merscwog There is an open issue for `minus` that overlaps with that scenario (see GROOVY-10964). Our users have been particularly desiring of an efficient minus which also caters for NumberAwareComparator. There is a PR (see https://github.com/apache/groovy/pull/1875) also for that but I haven't progressed that since the initial PR also is affected by the AbstractSet "bug". It looks like the JDK will fix that soon (see https://bugs.openjdk.org/browse/JDK-6394757). I was planning on updating the PR to try to align our behavior with how it will end up if possible. So, feel free to give that a crack if you want (i.e. incorporate GROOVY-10964), or ignore it altogether and skip test cases that are affected or add in a comment that the test case might change if GROOVY-10964 is resolved. -- 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: notifications-unsubscr...@groovy.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
