From 19e05359e641fff2fee410eda5572011926620a5 Mon Sep 17 00:00:00 2001 From: Kurt Partridge Date: Fri, 15 Feb 2013 12:21:16 -0800 Subject: [PATCH] [TestPrep3] Add helper method to retrive dictionary Change-Id: Id6e03c6f8a3e96979d589f0605ac056fcad7e1ff --- .../inputmethod/research/MainLogBuffer.java | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/java/src/com/android/inputmethod/research/MainLogBuffer.java b/java/src/com/android/inputmethod/research/MainLogBuffer.java index 3a87bf1df..182b924fc 100644 --- a/java/src/com/android/inputmethod/research/MainLogBuffer.java +++ b/java/src/com/android/inputmethod/research/MainLogBuffer.java @@ -93,6 +93,11 @@ public abstract class MainLogBuffer extends FixedLogBuffer { mSuggest = suggest; } + private Dictionary getDictionary() { + if (mSuggest == null || !mSuggest.hasMainDictionary()) return null; + return mSuggest.getMainDictionary(); + } + public void resetWordCounter() { mNumWordsUntilSafeToSample = mNumWordsBetweenNGrams; } @@ -137,18 +142,15 @@ public abstract class MainLogBuffer extends FixedLogBuffer { if (mNumWordsUntilSafeToSample > 0) { return false; } - if (mSuggest == null || !mSuggest.hasMainDictionary()) { + // Reload the dictionary in case it has changed (e.g., because the user has changed + // languages). + final Dictionary dictionary = getDictionary(); + if (dictionary == null) { // Main dictionary is unavailable. Since we cannot check it, we cannot tell if a // word is out-of-vocabulary or not. Therefore, we must judge the entire buffer // contents to potentially pose a privacy risk. return false; } - // Reload the dictionary in case it has changed (e.g., because the user has changed - // languages). - final Dictionary dictionary = mSuggest.getMainDictionary(); - if (dictionary == null) { - return false; - } // Check each word in the buffer. If any word poses a privacy threat, we cannot upload // the complete buffer contents in detail.