From 4e42e378fa41cfacf2c393a9428d75f6c6a0d7a8 Mon Sep 17 00:00:00 2001 From: Jean Chalard Date: Mon, 9 Jul 2012 17:35:47 +0900 Subject: [PATCH] Move stuff around for readability (A86) Change-Id: I5f4a18e43f3012acf96978d625325bf7c69c9c80 --- .../latin/ExpandableDictionary.java | 42 +++++++++---------- 1 file changed, 21 insertions(+), 21 deletions(-) diff --git a/java/src/com/android/inputmethod/latin/ExpandableDictionary.java b/java/src/com/android/inputmethod/latin/ExpandableDictionary.java index 76213c0da..05f3c0d94 100644 --- a/java/src/com/android/inputmethod/latin/ExpandableDictionary.java +++ b/java/src/com/android/inputmethod/latin/ExpandableDictionary.java @@ -263,6 +263,27 @@ public class ExpandableDictionary extends Dictionary { return suggestions; } + // @VisibleForTesting + boolean reloadDictionaryIfRequired() { + synchronized (mUpdatingLock) { + // If we need to update, start off a background task + if (mRequiresReload) startDictionaryLoadingTaskLocked(); + // Currently updating contacts, don't return any results. + return mUpdatingDictionary; + } + } + + @Override + public ArrayList getBigrams(final WordComposer codes, + final CharSequence previousWord) { + if (!reloadDictionaryIfRequired()) { + final ArrayList suggestions = new ArrayList(); + runBigramReverseLookUp(previousWord, suggestions); + return suggestions; + } + return null; + } + protected final ArrayList getWordsInner(final WordComposer codes, final CharSequence prevWordForBigrams, final ProximityInfo proximityInfo) { final ArrayList suggestions = new ArrayList(); @@ -589,16 +610,6 @@ public class ExpandableDictionary extends Dictionary { return searchWord(childNode.mChildren, word, depth + 1, childNode); } - // @VisibleForTesting - boolean reloadDictionaryIfRequired() { - synchronized (mUpdatingLock) { - // If we need to update, start off a background task - if (mRequiresReload) startDictionaryLoadingTaskLocked(); - // Currently updating contacts, don't return any results. - return mUpdatingDictionary; - } - } - private void runBigramReverseLookUp(final CharSequence previousWord, final ArrayList suggestions) { // Search for the lowercase version of the word only, because that's where bigrams @@ -610,17 +621,6 @@ public class ExpandableDictionary extends Dictionary { } } - @Override - public ArrayList getBigrams(final WordComposer codes, - final CharSequence previousWord) { - if (!reloadDictionaryIfRequired()) { - final ArrayList suggestions = new ArrayList(); - runBigramReverseLookUp(previousWord, suggestions); - return suggestions; - } - return null; - } - /** * Used for testing purposes and in the spell checker * This function will wait for loading from database to be done