[GROOVY-1995] minus(List self, Collection removeMe) throws IllegalStateException for dupplicate values

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0, 1.1-beta-1, 1.1-beta-2
• Fix Version/s: 1.1-rc-1
• Component/s: groovy-jdk
• Flags: Patch
• Assignee: Alexandru Popescu
• Reporter: Hauke Plambeck
• Created: 23/Jul/07 12:11
• Updated: 05/Apr/15 14:44
• Resolved: 05/Oct/07 16:16

Description

DefaultGroovyMethods.minus(List, Collection removeMe) and DefaultGroovyMethods.NumberAwareComparator have two problems.<br>

1. dupplicate values that are not Comparable cause IllegalStateExceptions
<code>
def a = new Object()
// throws IllegalStateException
assert [] == [ a ] - [ a, a ]
</code><br>
2. null values are not removed from a List
<code>
// fails
assert [] == [ null ] - [ null ]
</code><br>
I have attached a unit test and a fix based on the groovy-1.1-beta-2 source.