commit 23a5e9d1a78f42e76996e756fd1d6a37fc2ac679
author Jean Chalard <jchalard@google.com> Thu Mar 08 19:05:30 2012 -0800
committer Android (Google) Code Review <android-gerrit@google.com> Thu Mar 08 19:05:30 2012 -0800
tree c11a5e97c021d0b2144b92a6262d4515d3e78b61
parent 2621fe2256b68e59070d632dce72f35281bddf89
parent 82adfdaeae20b20eb24a5fe69eeefa37f36b4115

Merge "Make AutoCorrection static and non-instantiable (A5)"

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

diff --git a/java/src/com/android/inputmethod/latin/AutoCorrection.java b/java/src/com/android/inputmethod/latin/AutoCorrection.java
index 15b9f18..9754d15 100644
--- a/java/src/com/android/inputmethod/latin/AutoCorrection.java
+++ b/java/src/com/android/inputmethod/latin/AutoCorrection.java
@@ -26,7 +26,11 @@
     private static final boolean DBG = LatinImeLogger.sDBG;
     private static final String TAG = AutoCorrection.class.getSimpleName();
 
-    public static CharSequence updateAutoCorrectionStatus(Map<String, Dictionary> dictionaries,
+    private AutoCorrection() {
+        // Purely static class: can't instantiate.
+    }
+
+    public static CharSequence computeAutoCorrectionWord(Map<String, Dictionary> dictionaries,
             WordComposer wordComposer, ArrayList<CharSequence> suggestions, int[] sortedScores,
             CharSequence typedWord, double autoCorrectionThreshold, int correctionMode,
             CharSequence whitelistedWord) {

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 6cffae1..889d505 100644
--- a/java/src/com/android/inputmethod/latin/Suggest.java
+++ b/java/src/com/android/inputmethod/latin/Suggest.java
@@ -83,7 +83,6 @@
 
     private static final boolean DBG = LatinImeLogger.sDBG;
 
-    private AutoCorrection mAutoCorrection;
     private boolean mHasAutoCorrection;
 
     private Dictionary mMainDict;
@@ -125,7 +124,6 @@
     private void initWhitelistAndAutocorrectAndPool(final Context context, final Locale locale) {
         mWhiteListDictionary = new WhitelistDictionary(context, locale);
         addOrReplaceDictionary(mUnigramDictionaries, DICT_KEY_WHITELIST, mWhiteListDictionary);
-        mAutoCorrection = new AutoCorrection();
         StringBuilderPool.ensureCapacity(mPrefMaxSuggestions, getApproxMaxWordLength());
     }
 
@@ -273,7 +271,6 @@
             final WordComposer wordComposer, CharSequence prevWordForBigram,
             final ProximityInfo proximityInfo, final int correctionMode) {
         LatinImeLogger.onStartSuggestion(prevWordForBigram);
-        mAutoCorrection.init();
         mIsFirstCharCapitalized = wordComposer.isFirstCharCapitalized();
         mIsAllUpperCase = wordComposer.isAllUpperCase();
         mTrailingSingleQuotesCount = wordComposer.trailingSingleQuotesCount();
@@ -354,7 +351,7 @@
                 mWhiteListDictionary.getWhitelistedWord(consideredWordString));
 
         final CharSequence autoCorrection =
-                mAutoCorrection.updateAutoCorrectionStatus(mUnigramDictionaries, wordComposer,
+                AutoCorrection.computeAutoCorrectionWord(mUnigramDictionaries, wordComposer,
                 mSuggestions, mScores, consideredWord, mAutoCorrectionThreshold, correctionMode,
                 whitelistedWord);
         mHasAutoCorrection = (null != autoCorrection);