Merge "Pull up a common variable into the wrapping method (A50)"

main
Jean Chalard 2012-07-05 01:47:59 -07:00 committed by Android (Google) Code Review
commit 71d7fc91b7
1 changed files with 10 additions and 8 deletions

View File

@ -1704,17 +1704,19 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
return; return;
} }
final CharSequence typedWord;
if (isPredictions || !mWordComposer.isComposingWord()) { if (isPredictions || !mWordComposer.isComposingWord()) {
updateBigramPredictions(); typedWord = "";
updateBigramPredictions(typedWord);
} else { } else {
updateSuggestions(); typedWord = mWordComposer.getTypedWord();
updateSuggestions(typedWord);
} }
} }
private void updateSuggestions() { private void updateSuggestions(final CharSequence typedWord) {
// TODO: May need a better way of retrieving previous word // TODO: May need a better way of retrieving previous word
final CharSequence prevWord = mConnection.getPreviousWord(mCurrentSettings.mWordSeparators); final CharSequence prevWord = mConnection.getPreviousWord(mCurrentSettings.mWordSeparators);
final CharSequence typedWord = mWordComposer.getTypedWord();
// getSuggestedWords handles gracefully a null value of prevWord // getSuggestedWords handles gracefully a null value of prevWord
final SuggestedWords suggestedWords = mSuggest.getSuggestedWords(mWordComposer, final SuggestedWords suggestedWords = mSuggest.getSuggestedWords(mWordComposer,
prevWord, mKeyboardSwitcher.getKeyboard().getProximityInfo(), prevWord, mKeyboardSwitcher.getKeyboard().getProximityInfo(),
@ -1921,7 +1923,7 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
separatorCode, prevWord); separatorCode, prevWord);
} }
private void updateBigramPredictions() { private void updateBigramPredictions(final CharSequence typedWord) {
if (!mCurrentSettings.mBigramPredictionEnabled) { if (!mCurrentSettings.mBigramPredictionEnabled) {
setPunctuationSuggestions(); setPunctuationSuggestions();
return; return;
@ -1942,9 +1944,9 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
} }
if (null != suggestedWords && suggestedWords.size() > 0) { if (null != suggestedWords && suggestedWords.size() > 0) {
// Explicitly supply an empty typed word (the no-second-arg version of // Typed word is always empty. We pass it because the no-second-arg version of
// showSuggestions will retrieve the word near the cursor, we don't want that here) // showSuggestions will retrieve the word near the cursor, and we don't want that here
showSuggestions(suggestedWords, ""); showSuggestions(suggestedWords, typedWord);
} else { } else {
clearSuggestions(); clearSuggestions();
} }