Comparable comparator thows exceptions if either arg in the comparison is not
Comparable. However, the strings reverse which argument is incorrect,
claiming the first when it's o2, and the second when it's o1.
? .classpath
? .project
? .vcm_meta
? maven.log
? target
? src/java/org/apache/commons/collections/AbstractBag.java
Index: src/java/org/apache/commons/collections/comparators/ComparableComparator.java
===================================================================
RCS file: /home/cvspublic/jakarta-commons/collections/src/java/org/apache/commons/collections/comparators/ComparableComparator.java,v
retrieving revision 1.5
diff -u -w -r1.5 ComparableComparator.java
--- src/java/org/apache/commons/collections/comparators/ComparableComparator.java 12 Jun 2002 03:59:17 -0000 1.5
+++ src/java/org/apache/commons/collections/comparators/ComparableComparator.java 6 Oct 2002 01:29:16 -0000
@@ -121,7 +121,7 @@
} else {
// o2 wasn't comparable
throw new ClassCastException(
- "The first argument of this method was not a Comparable: " +
+ "The second argument of this method was not a Comparable: " +
o2.getClass().getName()
);
}
@@ -129,13 +129,13 @@
if(o2 instanceof Comparable) {
// o1 wasn't comparable
throw new ClassCastException(
- "The second argument of this method was not a Comparable: " +
+ "The first argument of this method was not a Comparable: " +
o1.getClass().getName()
);
} else {
// neither were comparable
throw new ClassCastException(
- "Both arguments of this method were not Comparables: " +
+ "Neither argument of this method was a Comparable: " +
o1.getClass().getName() + " and " + o2.getClass().getName()
);
}
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
