Seems mean to overwork the trusty String class by making it call toString on itself.
 
--------------------------------------
 
Index: java/src/org/apache/xindice/core/indexer/MemValueIndexer.java
===================================================================
RCS file: /home/cvspublic/xml-xindice/java/src/org/apache/xindice/core/indexer/MemValueIndexer.java,v
retrieving revision 1.9
diff -u -r1.9 MemValueIndexer.java
--- java/src/org/apache/xindice/core/indexer/MemValueIndexer.java 20 Jul 2004 20:31:37 -0000 1.9
+++ java/src/org/apache/xindice/core/indexer/MemValueIndexer.java 25 Aug 2004 02:35:46 -0000
@@ -974,7 +974,7 @@
                     //break;
                 case BOOLEAN:
                     // represented a a Byte of 0 (false) or 1 (true) so Comparable is implemented (true > false)
-                    return "[true][yes][1][y][on]".indexOf("[" + theValue.toString().toLowerCase() + "]") == -1 ? new Byte((byte) 0) : new Byte((byte) 1);
+                    return "[true][yes][1][y][on]".indexOf("[" + theValue.toLowerCase() + "]") == -1 ? new Byte((byte) 0) : new Byte((byte) 1);
                     //break;
             }
         } catch (Exception anException) {
Index: java/src/org/apache/xindice/core/indexer/ValueIndexer.java
===================================================================
RCS file: /home/cvspublic/xml-xindice/java/src/org/apache/xindice/core/indexer/ValueIndexer.java,v
retrieving revision 1.19
diff -u -r1.19 ValueIndexer.java
--- java/src/org/apache/xindice/core/indexer/ValueIndexer.java 20 Mar 2004 14:02:38 -0000 1.19
+++ java/src/org/apache/xindice/core/indexer/ValueIndexer.java 25 Aug 2004 02:36:26 -0000
@@ -221,9 +221,9 @@
                         b[1] = (byte) ((c >>> 0) & 0xFF);
                         break;
                     case BOOLEAN:
-                        if ("[true][yes][1][y][on]".indexOf("[" + value.toString().toLowerCase() + "]") != -1) {
+                        if ("[true][yes][1][y][on]".indexOf("[" + value.toLowerCase() + "]") != -1) {
                             b[0] = 1;
-                        } else if ("[false][no][0][n][off]".indexOf("[" + value.toString().toLowerCase() + "]") != -1) {
+                        } else if ("[false][no][0][n][off]".indexOf("[" + value.toLowerCase() + "]") != -1) {
                             b[0] = 0;
                         } else {
                             return EmptyValue;

Reply via email to