Merge "Fix an IOOB exception"

This commit is contained in:
Jean Chalard 2014-06-02 10:07:11 +00:00 committed by Android (Google) Code Review
commit 28538d916c

View File

@ -145,9 +145,12 @@ public final class WordComposer {
*/ */
public int copyCodePointsExceptTrailingSingleQuotesAndReturnCodePointCount( public int copyCodePointsExceptTrailingSingleQuotesAndReturnCodePointCount(
final int[] destination) { final int[] destination) {
// This method can be called on a separate thread and mTypedWordCache can change while we
// are executing this method.
final String typedWord = mTypedWordCache.toString();
// lastIndex is exclusive // lastIndex is exclusive
final int lastIndex = mTypedWordCache.length() final int lastIndex = typedWord.length()
- StringUtils.getTrailingSingleQuotesCount(mTypedWordCache); - StringUtils.getTrailingSingleQuotesCount(typedWord);
if (lastIndex <= 0) { if (lastIndex <= 0) {
// The string is empty or contains only single quotes. // The string is empty or contains only single quotes.
return 0; return 0;
@ -155,11 +158,11 @@ public final class WordComposer {
// The following function counts the number of code points in the text range which begins // The following function counts the number of code points in the text range which begins
// at index 0 and extends to the character at lastIndex. // at index 0 and extends to the character at lastIndex.
final int codePointSize = Character.codePointCount(mTypedWordCache, 0, lastIndex); final int codePointSize = Character.codePointCount(typedWord, 0, lastIndex);
if (codePointSize > destination.length) { if (codePointSize > destination.length) {
return -1; return -1;
} }
return StringUtils.copyCodePointsAndReturnCodePointCount(destination, mTypedWordCache, 0, return StringUtils.copyCodePointsAndReturnCodePointCount(destination, typedWord, 0,
lastIndex, true /* downCase */); lastIndex, true /* downCase */);
} }