You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2018/09/27 21:48:00 UTC
svn commit: r1842178 -
/commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/language/ColognePhonetic.java
Author: sebb
Date: Thu Sep 27 21:47:59 2018
New Revision: 1842178
URL: http://svn.apache.org/viewvc?rev=1842178&view=rev
Log:
Simplify - don't cache dynamic length
Modified:
commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/language/ColognePhonetic.java
Modified: commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/language/ColognePhonetic.java
URL: http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/language/ColognePhonetic.java?rev=1842178&r1=1842177&r2=1842178&view=diff
==============================================================================
--- commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/language/ColognePhonetic.java (original)
+++ commons/proper/codec/trunk/src/main/java/org/apache/commons/codec/language/ColognePhonetic.java Thu Sep 27 21:47:59 2018
@@ -325,12 +325,10 @@ public class ColognePhonetic implements
char code;
char chr;
- int rightLength = input.length();
-
- while (rightLength > 0) {
+ while (input.length() > 0) {
chr = input.removeNext();
- if ((rightLength = input.length()) > 0) {
+ if (input.length() > 0) {
nextChar = input.getNextChar();
} else {
nextChar = '-';
@@ -354,7 +352,6 @@ public class ColognePhonetic implements
} else if (chr == 'X' && !arrayContains(CKQ, lastChar)) {
code = '4';
input.addLeft('S');
- rightLength++;
} else if (chr == 'S' || chr == 'Z') {
code = '8';
} else if (chr == 'C') {