commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Goran Hacek (JIRA)" <>
Subject [jira] Commented: (COLLECTIONS-340) NOPClosure has a broken equals method
Date Wed, 14 Oct 2009 11:50:31 GMT


Goran Hacek commented on COLLECTIONS-340:

None of the classes in "org.apache.commons.collections.functors" package implement equals()
or hashCode() so why should NOPClosure be any different.

I suggest removal of both equals() and hashCode()  from NOPClosure.

> NOPClosure has a broken equals method
> -------------------------------------
>                 Key: COLLECTIONS-340
>                 URL:
>             Project: Commons Collections
>          Issue Type: Bug
>          Components: Functor
>            Reporter: Goran Hacek
>            Priority: Minor
>             Fix For: Generics
> Equals method of NOPClosure class doesn't follow objects equals contract. It isn't symmetric
nor transitive and it is just plain wrong.
> Following code evaluates to true on most (if not all) JVM implementations:
> {noformat} NOPClosure.INSTANCE.equals(new Integer(System.identityHashCode(NOPClosure.INSTANCE)))

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message