You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2014/04/10 14:51:29 UTC

svn commit: r1586286 - /commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/language/Metaphone.java

Author: ggregory
Date: Thu Apr 10 12:51:29 2014
New Revision: 1586286

URL: http://svn.apache.org/r1586286
Log:
Refactor API calls in lvar.

Modified:
    commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/language/Metaphone.java

Modified: commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/language/Metaphone.java
URL: http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/language/Metaphone.java?rev=1586286&r1=1586285&r2=1586286&view=diff
==============================================================================
--- commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/language/Metaphone.java (original)
+++ commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/language/Metaphone.java Thu Apr 10 12:51:29 2014
@@ -91,11 +91,12 @@ public class Metaphone implements String
      */
     public String metaphone(final String txt) {
         boolean hard = false;
-        if (txt == null || txt.length() == 0) {
+        final int txtLength = txt.length();
+        if (txt == null || txtLength == 0) {
             return "";
         }
         // single character is itself
-        if (txt.length() == 1) {
+        if (txtLength == 1) {
             return txt.toUpperCase(java.util.Locale.ENGLISH);
         }