Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c03bd090 authored by Jean Chalard's avatar Jean Chalard Committed by Android (Google) Code Review
Browse files

Merge "Cleanup, part 4 (C4)"

parents bd765d33 9b233ece
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -1712,7 +1712,7 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
        // getSuggestedWords handles gracefully a null value of prevWord
        final SuggestedWords suggestedWords = mSuggest.getSuggestedWords(mWordComposer,
                prevWord, mKeyboardSwitcher.getKeyboard().getProximityInfo(),
                mCurrentSettings.mCorrectionMode);
                mCurrentSettings.isCorrectionOn());

        // Basically, we update the suggestion strip only when suggestion count > 1.  However,
        // there is an exception: We update the suggestion strip whenever typed word's length
+5 −3
Original line number Diff line number Diff line
@@ -207,9 +207,11 @@ public class SettingsValues {
    }

    private int createCorrectionMode() {
        final boolean shouldAutoCorrect = mAutoCorrectEnabled
                && !mInputAttributes.mInputTypeNoAutoCorrect;
        return shouldAutoCorrect ? Suggest.CORRECTION_FULL : Suggest.CORRECTION_NONE;
        if (mAutoCorrectEnabled && !mInputAttributes.mInputTypeNoAutoCorrect) {
            return Suggest.CORRECTION_FULL;
        } else {
            return Suggest.CORRECTION_NONE;
        }
    }

    private int createSuggestionVisibility(final Resources res) {
+12 −4
Original line number Diff line number Diff line
@@ -249,10 +249,18 @@ public class Suggest implements Dictionary.WordCallback {
                true /* isPrediction */);
    }

    // TODO: cleanup dictionaries looking up and suggestions building with SuggestedWords.Builder
    // Wrapper for test compatibility. TODO: remove this method
    public SuggestedWords getSuggestedWords(
            final WordComposer wordComposer, CharSequence prevWordForBigram,
            final ProximityInfo proximityInfo, final int correctionMode) {
        return getSuggestedWords(wordComposer, prevWordForBigram, proximityInfo,
                Suggest.CORRECTION_FULL == correctionMode);
    }

    // TODO: cleanup dictionaries looking up and suggestions building with SuggestedWords.Builder
    public SuggestedWords getSuggestedWords(
            final WordComposer wordComposer, CharSequence prevWordForBigram,
            final ProximityInfo proximityInfo, final boolean isCorrectionEnabled) {
        LatinImeLogger.onStartSuggestion(prevWordForBigram);
        mIsFirstCharCapitalized = wordComposer.isFirstCharCapitalized();
        mIsAllUpperCase = wordComposer.isAllUpperCase();
@@ -267,7 +275,7 @@ public class Suggest implements Dictionary.WordCallback {
        LatinImeLogger.onAddSuggestedWord(typedWord, Suggest.DIC_USER_TYPED, Dictionary.UNIGRAM);
        mConsideredWord = consideredWord;

        if (wordComposer.size() <= 1 && (correctionMode == CORRECTION_FULL)) {
        if (wordComposer.size() <= 1 && isCorrectionEnabled) {
            // At first character typed, search only the bigrams
            mBigramSuggestions = new ArrayList<SuggestedWordInfo>(PREF_MAX_BIGRAMS);

@@ -324,7 +332,7 @@ public class Suggest implements Dictionary.WordCallback {
                mIsFirstCharCapitalized, mWhiteListDictionary.getWhitelistedWord(consideredWord));

        final boolean hasAutoCorrection;
        if (CORRECTION_FULL == correctionMode) {
        if (isCorrectionEnabled) {
            final CharSequence autoCorrection =
                    AutoCorrection.computeAutoCorrectionWord(mUnigramDictionaries, wordComposer,
                            mSuggestions, consideredWord, mAutoCorrectionThreshold,
@@ -374,7 +382,7 @@ public class Suggest implements Dictionary.WordCallback {
                && hasMainDictionary();

        boolean autoCorrectionAvailable = hasAutoCorrection;
        if (correctionMode == CORRECTION_FULL) {
        if (isCorrectionEnabled) {
            autoCorrectionAvailable |= !allowsToBeAutoCorrected;
        }
        // Don't auto-correct words with multiple capital letter