nyancrimew/LatinIME
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Fix an OOB

Browse source 
The combination of I6b0262fe which removes the internal
size() > 1 test in the safety net calculation and I642a5893
which assumed DBG means there is at least one suggestion.
This change writes the test correctly.

Change-Id: Ib63bce4e163a785f408dd145c23d64cd58fa8c90
This commit is contained in:
Jean Chalard 2012-03-12 16:50:58 +09:00
parent 30936116d8
commit c0c20b386f
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
java/src/com/android/inputmethod/latin/suggestions/SuggestionsView.java
View file

@ -329,7 +329,7 @@ public class SuggestionsView extends RelativeLayout implements OnClickListener,
} else { } else {
color = mColorTypedWord; color = mColorTypedWord;
} }
if (LatinImeLogger.sDBG) { if (LatinImeLogger.sDBG && suggestedWords.size() > 1) {
// If we auto-correct, then the autocorrection is in slot 0 and the typed word // If we auto-correct, then the autocorrection is in slot 0 and the typed word
// is in slot 1. // is in slot 1.
if (index == mCenterSuggestionIndex && suggestedWords.mHasAutoCorrectionCandidate if (index == mCenterSuggestionIndex && suggestedWords.mHasAutoCorrectionCandidate