[CODEC-174] Small (4-5%) but consistent performance gain with this patch from https://issues.apache.org/jira/secure/attachment/12613346/CODEC-174-refactor-join-method-in-Phoneme.patch

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

Author: garydgregory

src/main/java/org/apache/commons/codec/language/bm/PhoneticEngine.java

@@ -30,6 +30,9 @@
 import java.util.Set;
 import java.util.TreeSet;
 
+import org.apache.commons.codec.language.bm.Languages.LanguageSet;
+import org.apache.commons.codec.language.bm.Rule.Phoneme;
+
 /**
  * Converts words into potential phonetic representations.
  * <p>
@@ -107,8 +110,9 @@ public PhonemeBuilder apply(final Rule.PhonemeExpr phonemeExpr, final int maxPho
 
             EXPR: for (final Rule.Phoneme left : this.phonemes) {
                 for (final Rule.Phoneme right : phonemeExpr.getPhonemes()) {
-                    final Rule.Phoneme join = left.join(right);
-                    if (!join.getLanguages().isEmpty()) {
+                	LanguageSet languages = left.getLanguages().restrictTo(right.getLanguages());
+                	if (!languages.isEmpty()) {
+                		final Rule.Phoneme join = new Phoneme(left, right, languages);
                         if (newPhonemes.size() < maxPhonemes) {
                             newPhonemes.add(join);
                         } else {

src/main/java/org/apache/commons/codec/language/bm/Rule.java

@@ -110,9 +110,13 @@ public Phoneme(final CharSequence phonemeText, final Languages.LanguageSet langu
         }
 
         public Phoneme(final Phoneme phonemeLeft, final Phoneme phonemeRight) {
-            this.phonemeText = new StringBuilder(phonemeLeft.phonemeText);
+            this(phonemeLeft.phonemeText, phonemeLeft.languages);
+            this.phonemeText.append(phonemeRight.phonemeText);
+        }
+
+        public Phoneme(final Phoneme phonemeLeft, final Phoneme phonemeRight, final Languages.LanguageSet languages) {
+            this(phonemeLeft.phonemeText, languages);
             this.phonemeText.append(phonemeRight.phonemeText);
-            this.languages = phonemeLeft.languages;
         }
 
         public Phoneme  append(final CharSequence str) {
@@ -132,11 +136,6 @@ public Iterable<Phoneme> getPhonemes() {
         public CharSequence getPhonemeText() {
             return this.phonemeText;
         }
-
-        public Phoneme join(final Phoneme right) {
-            return new Phoneme(this.phonemeText.toString() + right.phonemeText.toString(),
-                               this.languages.restrictTo(right.languages));
-        }
     }
 
     public interface PhonemeExpr {