Yoav, Your suggestion is a good one. Not exactly sure how to work out the <> part of the comparision, but it would be one less special purpose method.
I'll think about it. thanks, -Mark > -----Original Message----- > From: Shapira, Yoav [mailto:[EMAIL PROTECTED] > Sent: Monday, October 20, 2003 5:40 AM > To: Log4J Developers List > Subject: RE: Plugins, Receivers, Watchdogs > > > > Howdy, > > >for some time. Mainly, getting rid of the overriding of the equals() > >method > >to determing if 2 instances of a plugin are equivalent. > Doing this is > not > >so great and has implications to override hashCode(), which > requires a > lot > >more work than it was worth. There is now an isEquivalent() > method in > the > >Plugin interface that can be implemented to determine, to the degree > the > > I agree about overriding equals/hashCode, but there is an approaches > that might be better than what you've done: Implement > Comparable, i.e. > the compareTo method. This is the standard thing to do when > you want to > compare things and not implement equals. You can keep ifEquivalent if > you'd like and have it do compareTo() == 0. > > Implementing Comparable has other freebie benefits, from everything in > java that can handle an array or collection of Comparable objects: > sorting, searching, eliminating duplicates, etc. > > That said, the above is general: I haven't look at the plugin case > specifically. > > Yoav Shapira > > > > This e-mail, including any attachments, is a confidential > business communication, and may contain information that is > confidential, proprietary and/or privileged. This e-mail is > intended only for the individual(s) to whom it is addressed, > and may not be saved, copied, printed, disclosed or used by > anyone else. If you are not the(an) intended recipient, > please immediately delete this e-mail from your computer > system and notify the sender. Thank you. > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
