From 7ad088a4cf5593a2c72e143f10ba2e7ffed8bb09 Mon Sep 17 00:00:00 2001 From: Jean Chalard Date: Thu, 28 Jun 2012 20:16:44 +0900 Subject: [PATCH] Remove useless code (A42) If allowsAutoCorrected is false, there is no point in making hasAutoCorrection true, since in the only place where we use it again, it's &&'ed with allowsAutoCorrected ! Well that was extremely obscure, good thing refactoring allowed to realize this was useless >.> Change-Id: I34936d445f1ced17c7bd04a9524bf608f9e8b9c8 --- java/src/com/android/inputmethod/latin/Suggest.java | 5 ----- 1 file changed, 5 deletions(-) diff --git a/java/src/com/android/inputmethod/latin/Suggest.java b/java/src/com/android/inputmethod/latin/Suggest.java index 398881335..9e9ffc76a 100644 --- a/java/src/com/android/inputmethod/latin/Suggest.java +++ b/java/src/com/android/inputmethod/latin/Suggest.java @@ -240,11 +240,6 @@ public class Suggest { hasAutoCorrection = false; } else if (null != whitelistedWord) { hasAutoCorrection = true; - } else if (!allowsToBeAutoCorrected) { - // TODO: make the variable name clearer. If we don't allow auto-correct, that means - // this word is a dictionary word that is not whitelisted, so it should auto-correct - // to itself! Hence, the true here. - hasAutoCorrection = true; } else if (suggestionsSet.isEmpty()) { hasAutoCorrection = false; } else if (AutoCorrection.suggestionExceedsAutoCorrectionThreshold(suggestionsSet.first(),