commit 3ecb2747b3b46991a0fb8e1573d689611ecbd1cc
author Jean Chalard <jchalard@google.com> Wed Jun 27 18:28:35 2012 +0900
committer Jean Chalard <jchalard@google.com> Wed Jun 27 18:55:43 2012 +0900
tree d454ec6c45ae625fb40f5e06a31eaf8c443e0dd6
parent 3705ef407b654e818d4ec8854f08495bca4688d9

Optimization.

It's useless to do this for all suggestions including those
that will be ousted from the list.

Change-Id: I7bd63d783e21984a269e4b1ae17e9b9d8c0859e3

java/src/com/android/inputmethod/latin/Suggest.java

diff --git a/java/src/com/android/inputmethod/latin/Suggest.java b/java/src/com/android/inputmethod/latin/Suggest.java
index 389d4e2..de05ed6 100644
--- a/java/src/com/android/inputmethod/latin/Suggest.java
+++ b/java/src/com/android/inputmethod/latin/Suggest.java
@@ -250,6 +250,15 @@
             }
         }
 
+        for (int i = 0; i < suggestionsContainer.size(); ++i) {
+            final SuggestedWordInfo wordInfo = suggestionsContainer.get(i);
+            final SuggestedWordInfo transformedWordInfo = getTransformedSuggestedWordInfo(wordInfo,
+                    mLocale, mIsAllUpperCase, mIsFirstCharCapitalized, mTrailingSingleQuotesCount);
+            suggestionsContainer.set(i, transformedWordInfo);
+            LatinImeLogger.onAddSuggestedWord(transformedWordInfo.mWord.toString(),
+                    transformedWordInfo.mSourceDict);
+        }
+
         final CharSequence whitelistedWord = capitalizeWord(mIsAllUpperCase,
                 mIsFirstCharCapitalized, mWhiteListDictionary.getWhitelistedWord(consideredWord));
 
@@ -370,7 +379,7 @@
     private static final SuggestedWordInfoComparator sSuggestedWordInfoComparator =
             new SuggestedWordInfoComparator();
 
-    public void addWord(final SuggestedWordInfo wordInfo,
+    private static void addWord(final SuggestedWordInfo wordInfo,
             final ArrayList<SuggestedWordInfo> suggestions) {
         final int index =
                 Collections.binarySearch(suggestions, wordInfo, sSuggestedWordInfoComparator);
@@ -378,15 +387,10 @@
         // it returns -insertionPoint - 1. We want the insertion point, so:
         final int pos = index >= 0 ? index : -index - 1;
         if (pos >= MAX_SUGGESTIONS) return;
-
-        final SuggestedWordInfo transformedWordInfo = getTransformedSuggestedWordInfo(wordInfo,
-                mLocale, mIsAllUpperCase, mIsFirstCharCapitalized, mTrailingSingleQuotesCount);
-        suggestions.add(pos, transformedWordInfo);
+        suggestions.add(pos, wordInfo);
         if (suggestions.size() > MAX_SUGGESTIONS) {
             suggestions.remove(MAX_SUGGESTIONS);
         }
-        LatinImeLogger.onAddSuggestedWord(transformedWordInfo.mWord.toString(),
-                transformedWordInfo.mSourceDict);
     }
 
     private static SuggestedWordInfo getTransformedSuggestedWordInfo(