am 5252ca77: am cd0e0048: Merge "Ensure #offer always actually offers something." into jb-mr1-dev
* commit '5252ca773f9f5ce1ddd8afe423850c64834ba1a0': Ensure #offer always actually offers something.main
commit
a01d0212d6
|
@ -194,7 +194,7 @@ public abstract class AndroidWordLevelSpellCheckerSession extends Session {
|
|||
DictAndProximity dictInfo = null;
|
||||
try {
|
||||
dictInfo = mDictionaryPool.pollWithDefaultTimeout();
|
||||
if (null == dictInfo) {
|
||||
if (!DictionaryPool.isAValidDictionary(dictInfo)) {
|
||||
return AndroidSpellCheckerService.getNotInDictEmptySuggestions();
|
||||
}
|
||||
return dictInfo.mDictionary.isValidWord(inText)
|
||||
|
@ -237,7 +237,7 @@ public abstract class AndroidWordLevelSpellCheckerSession extends Session {
|
|||
DictAndProximity dictInfo = null;
|
||||
try {
|
||||
dictInfo = mDictionaryPool.pollWithDefaultTimeout();
|
||||
if (null == dictInfo) {
|
||||
if (!DictionaryPool.isAValidDictionary(dictInfo)) {
|
||||
return AndroidSpellCheckerService.getNotInDictEmptySuggestions();
|
||||
}
|
||||
final ArrayList<SuggestedWordInfo> suggestions =
|
||||
|
|
|
@ -18,6 +18,12 @@ package com.android.inputmethod.latin.spellcheck;
|
|||
|
||||
import android.util.Log;
|
||||
|
||||
import com.android.inputmethod.keyboard.ProximityInfo;
|
||||
import com.android.inputmethod.latin.Dictionary;
|
||||
import com.android.inputmethod.latin.SuggestedWords.SuggestedWordInfo;
|
||||
import com.android.inputmethod.latin.WordComposer;
|
||||
|
||||
import java.util.ArrayList;
|
||||
import java.util.Locale;
|
||||
import java.util.concurrent.LinkedBlockingQueue;
|
||||
import java.util.concurrent.TimeUnit;
|
||||
|
@ -39,6 +45,26 @@ public class DictionaryPool extends LinkedBlockingQueue<DictAndProximity> {
|
|||
private final Locale mLocale;
|
||||
private int mSize;
|
||||
private volatile boolean mClosed;
|
||||
final static ArrayList<SuggestedWordInfo> noSuggestions = new ArrayList<SuggestedWordInfo>();
|
||||
private final static DictAndProximity dummyDict = new DictAndProximity(
|
||||
new Dictionary(Dictionary.TYPE_MAIN) {
|
||||
@Override
|
||||
public ArrayList<SuggestedWordInfo> getSuggestions(final WordComposer composer,
|
||||
final CharSequence prevWord, final ProximityInfo proximityInfo) {
|
||||
return noSuggestions;
|
||||
}
|
||||
@Override
|
||||
public boolean isValidWord(CharSequence word) {
|
||||
// This is never called. However if for some strange reason it ever gets
|
||||
// called, returning true is less destructive (it will not underline the
|
||||
// word in red).
|
||||
return true;
|
||||
}
|
||||
}, null);
|
||||
|
||||
static public boolean isAValidDictionary(final DictAndProximity dictInfo) {
|
||||
return null != dictInfo && dummyDict != dictInfo;
|
||||
}
|
||||
|
||||
public DictionaryPool(final int maxSize, final AndroidSpellCheckerService service,
|
||||
final Locale locale) {
|
||||
|
@ -98,7 +124,7 @@ public class DictionaryPool extends LinkedBlockingQueue<DictAndProximity> {
|
|||
public boolean offer(final DictAndProximity dict) {
|
||||
if (mClosed) {
|
||||
dict.mDictionary.close();
|
||||
return false;
|
||||
return super.offer(dummyDict);
|
||||
} else {
|
||||
return super.offer(dict);
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue