> IllegalArgumentException is a bit weird here in this context. While I > agree it it probably the most appropriate exception when the arguments > aren't comparable, the Comparator javadoc declares it throws a > ClassCastException "if the arguments' types prevent it from being > compared". I'm ok with the IllegalArgument, but you should still add > javadoc to declare it since it isn't declared in the Comparator's API > docs.
Ack. You're right. Change made to ClassCastException. -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
