Allow adding DICTIONARY_MAX_WORD_LENGTH-length words.
Bug: 12725817 Change-Id: Ib9b30473ba94242a7b97ccd3b850068f596a5d17
This commit is contained in:
parent
3225b6fe66
commit
d9a8f2a82f
4 changed files with 6 additions and 7 deletions
|
@ -231,7 +231,7 @@ public class ContactsBinaryDictionary extends ExpandableBinaryDictionary {
|
|||
// Don't add single letter words, possibly confuses
|
||||
// capitalization of i.
|
||||
final int wordLen = StringUtils.codePointCount(word);
|
||||
if (wordLen < MAX_WORD_LENGTH && wordLen > 1) {
|
||||
if (wordLen <= MAX_WORD_LENGTH && wordLen > 1) {
|
||||
if (DEBUG) {
|
||||
Log.d(TAG, "addName " + name + ", " + word + ", " + prevWordsInfo);
|
||||
}
|
||||
|
|
|
@ -253,12 +253,12 @@ public class UserBinaryDictionary extends ExpandableBinaryDictionary {
|
|||
final int frequency = cursor.getInt(indexFrequency);
|
||||
final int adjustedFrequency = scaleFrequencyFromDefaultToLatinIme(frequency);
|
||||
// Safeguard against adding really long words.
|
||||
if (word.length() < MAX_WORD_LENGTH) {
|
||||
if (word.length() <= MAX_WORD_LENGTH) {
|
||||
runGCIfRequiredLocked(true /* mindsBlockByGC */);
|
||||
addUnigramLocked(word, adjustedFrequency, null /* shortcutTarget */,
|
||||
0 /* shortcutFreq */, false /* isNotAWord */,
|
||||
false /* isBlacklisted */, BinaryDictionary.NOT_A_VALID_TIMESTAMP);
|
||||
if (null != shortcut && shortcut.length() < MAX_WORD_LENGTH) {
|
||||
if (null != shortcut && shortcut.length() <= MAX_WORD_LENGTH) {
|
||||
runGCIfRequiredLocked(true /* mindsBlockByGC */);
|
||||
addUnigramLocked(shortcut, adjustedFrequency, word,
|
||||
USER_DICT_SHORTCUT_FREQUENCY, true /* isNotAWord */,
|
||||
|
|
|
@ -62,8 +62,8 @@ public class UserHistoryDictionary extends DecayingExpandableBinaryDictionaryBas
|
|||
final PrevWordsInfo prevWordsInfo, final String word, final boolean isValid,
|
||||
final int timestamp, final DistracterFilter distracterFilter) {
|
||||
final CharSequence prevWord = prevWordsInfo.mPrevWordsInfo[0].mWord;
|
||||
if (word.length() >= Constants.DICTIONARY_MAX_WORD_LENGTH ||
|
||||
(prevWord != null && prevWord.length() >= Constants.DICTIONARY_MAX_WORD_LENGTH)) {
|
||||
if (word.length() > Constants.DICTIONARY_MAX_WORD_LENGTH ||
|
||||
(prevWord != null && prevWord.length() > Constants.DICTIONARY_MAX_WORD_LENGTH)) {
|
||||
return;
|
||||
}
|
||||
final int frequency = isValid ?
|
||||
|
|
|
@ -386,8 +386,7 @@ public class DictionaryInfoUtils {
|
|||
final SpacingAndPunctuations spacingAndPunctuations) {
|
||||
if (TextUtils.isEmpty(text)) return false;
|
||||
final int length = text.length();
|
||||
// TODO: Make this test "length > Constants.DICTIONARY_MAX_WORD_LENGTH".
|
||||
if (length >= Constants.DICTIONARY_MAX_WORD_LENGTH) {
|
||||
if (length > Constants.DICTIONARY_MAX_WORD_LENGTH) {
|
||||
return false;
|
||||
}
|
||||
int i = 0;
|
||||
|
|
Loading…
Reference in a new issue