The StringEncoderComparator used to call o1.toString(), and

o2.toString() on the parameters supplied to the compare
method.  There is no need to require that only Strings
are passed to the StringEncoderComparator - (actually
there is no reason why we couldn't have a generic Encoder
comparator.  This patch is the first step towards a more
generic object, and it allows us to test the exception condition
in this compare() method.


git-svn-id: https://svn.apache.org/repos/asf/jakarta/commons/proper/codec/trunk@130206 13f79535-47bb-0310-9956-ffa450edef68

Author: Timothy O'Brien

src/java/org/apache/commons/codec/StringEncoderComparator.java

@@ -70,7 +70,7 @@
  *
  * @author Tim O'Brien
  * @author Gary Gregory
- * @version $Id: StringEncoderComparator.java,v 1.6 2003/10/05 21:45:48 tobrien Exp $
+ * @version $Id: StringEncoderComparator.java,v 1.7 2003/10/12 01:34:06 tobrien Exp $
  */
 public class StringEncoderComparator implements Comparator {
 
@@ -103,8 +103,8 @@ public int compare(Object o1, Object o2) {
         int compareCode = 0;
 
         try {
-            String s1 = stringEncoder.encode(o1.toString());
-            String s2 = stringEncoder.encode(o2.toString());
+            Comparable s1 = (Comparable) ((Encoder) stringEncoder).encode(o1);
+            Comparable s2 = (Comparable) ((Encoder) stringEncoder).encode(o2);
             compareCode = s1.compareTo(s2);
         } 
         catch (EncoderException ee) {