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: I34936d445f1ced17c7bd04a9524bf608f9e8b9c8main
parent
90d300c770
commit
7ad088a4cf
|
@ -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(),
|
||||
|
|
Loading…
Reference in New Issue