commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bay...@apache.org
Subject svn commit: r589052 - in /commons/proper/codec/trunk/src: java/org/apache/commons/codec/language/RefinedSoundex.java test/org/apache/commons/codec/language/RefinedSoundexTest.java
Date Sat, 27 Oct 2007 06:48:27 GMT
Author: bayard
Date: Fri Oct 26 23:48:26 2007
New Revision: 589052

URL: http://svn.apache.org/viewvc?rev=589052&view=rev
Log:
Applying Sebb's fix from CODEC-56, along with a test

Modified:
    commons/proper/codec/trunk/src/java/org/apache/commons/codec/language/RefinedSoundex.java
    commons/proper/codec/trunk/src/test/org/apache/commons/codec/language/RefinedSoundexTest.java

Modified: commons/proper/codec/trunk/src/java/org/apache/commons/codec/language/RefinedSoundex.java
URL: http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/java/org/apache/commons/codec/language/RefinedSoundex.java?rev=589052&r1=589051&r2=589052&view=diff
==============================================================================
--- commons/proper/codec/trunk/src/java/org/apache/commons/codec/language/RefinedSoundex.java
(original)
+++ commons/proper/codec/trunk/src/java/org/apache/commons/codec/language/RefinedSoundex.java
Fri Oct 26 23:48:26 2007
@@ -30,13 +30,7 @@
  */
 public class RefinedSoundex implements StringEncoder {
 
-    /**
-	 * This static variable contains an instance of the RefinedSoundex using
-	 * the US_ENGLISH mapping.
-	 */
-    public static final RefinedSoundex US_ENGLISH = new RefinedSoundex();
-
-    /**
+   /**
 	 * RefinedSoundex is *refined* for a number of reasons one being that the
 	 * mappings have been altered. This implementation contains default
 	 * mappings for US English.
@@ -51,6 +45,12 @@
     private final char[] soundexMapping;
 
     /**
+	 * This static variable contains an instance of the RefinedSoundex using
+	 * the US_ENGLISH mapping.
+	 */
+    public static final RefinedSoundex US_ENGLISH = new RefinedSoundex();
+
+     /**
 	 * Creates an instance of the RefinedSoundex object using the default US
 	 * English mapping.
 	 */

Modified: commons/proper/codec/trunk/src/test/org/apache/commons/codec/language/RefinedSoundexTest.java
URL: http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/test/org/apache/commons/codec/language/RefinedSoundexTest.java?rev=589052&r1=589051&r2=589052&view=diff
==============================================================================
--- commons/proper/codec/trunk/src/test/org/apache/commons/codec/language/RefinedSoundexTest.java
(original)
+++ commons/proper/codec/trunk/src/test/org/apache/commons/codec/language/RefinedSoundexTest.java
Fri Oct 26 23:48:26 2007
@@ -104,10 +104,13 @@
         assertEquals("T60", this.getEncoder().encode("the"));
         assertEquals("L7050", this.getEncoder().encode("lazy"));
         assertEquals("D6043", this.getEncoder().encode("dogs"));
+
+        // Testing CODEC-56
+        assertEquals("D6043", RefinedSoundex.US_ENGLISH.encode("dogs"));
     }
 
 	public void testGetMappingCodeNonLetter() {
 		char code = this.getEncoder().getMappingCode('#');
 		assertEquals("Code does not equals zero", 0, code);
 	}
-}
\ No newline at end of file
+}



Mime
View raw message