Not that it really makes any difference because you can't state that the comparators are equivalent easily :-)
But it might make sense to get rid of that method, so that at least it returns true if if the instances are the same. But all instances of the same class should be the same, right? Maybe it would make sense to have return 'other != null && this.getClass() == other.getClass()'. (Unless there's some sort of sort parameter in the constructor... in which case the default Object method would make sense). So, whilst you may not be able to find other instances that are true, it should at least return true for itself ... Alex. On 13/01/07, Nathan Beyer <[EMAIL PROTECTED]> wrote:
Assuming the class is actually a Comparator, that's probably fine, though it does seem a little weird. A Comparator is supposed to implement an equals method to determine if two Comparators are logical equal such that given two different instances they would both sort something exactly the same way. Perhaps this class is just saying that can never be possible. -Nathan On 1/12/07, Sian January <[EMAIL PROTECTED]> wrote: > Hello, > > I have found a class in the security module that overrides the equals > methods, but just returns false within that method. There doesn't seem to > be any comment about why it does this so I was just wondering if anyone had > any idea? The class is > org.apache.harmony.security.x501.AttributeTypeAndValueComparator. > > Thanks, > > Sian > > -- > Sian January > > IBM Java Technology Centre, UK > >
