diff options
| author | Gilles Debunne <debunne@google.com> | 2011-04-08 11:31:25 -0700 |
|---|---|---|
| committer | Gilles Debunne <debunne@google.com> | 2011-04-08 11:33:23 -0700 |
| commit | daa4a95a54909d53a325eb06ca22130743b5be04 (patch) | |
| tree | b3d058e678d11df82b2a44d4f5efcbac875d4ea0 /core/java/android/text/method | |
| parent | 99eefad9ab6f573641f45a950b03bf5e7766506d (diff) | |
| download | frameworks_base-daa4a95a54909d53a325eb06ca22130743b5be04.zip frameworks_base-daa4a95a54909d53a325eb06ca22130743b5be04.tar.gz frameworks_base-daa4a95a54909d53a325eb06ca22130743b5be04.tar.bz2 | |
Fix in BreakIterator.
An index equal to the length of text is indeed a valid position,
representing a cursor located after the last character.
Also changed iterator to comply to the documentation.
Change-Id: I7b9e427e4e37df20ab7e66d99053a9d8a6af6eb3
Diffstat (limited to 'core/java/android/text/method')
| -rw-r--r-- | core/java/android/text/method/ArrowKeyMovementMethod.java | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/core/java/android/text/method/ArrowKeyMovementMethod.java b/core/java/android/text/method/ArrowKeyMovementMethod.java index 80c0106..b25ba8d 100644 --- a/core/java/android/text/method/ArrowKeyMovementMethod.java +++ b/core/java/android/text/method/ArrowKeyMovementMethod.java @@ -390,7 +390,7 @@ public class ArrowKeyMovementMethod extends BaseMovementMethod implements Moveme } private boolean isValidOffset(int offset) { - return offset >= 0 && offset < mCurrent.length(); + return offset >= 0 && offset <= mCurrent.length(); } private boolean isLetterOrDigit(int offset) { @@ -404,7 +404,7 @@ public class ArrowKeyMovementMethod extends BaseMovementMethod implements Moveme /** {@inheritDoc} */ public int preceding(int offset) { // always round cursor index into valid string index - offset = MathUtils.constrain(offset, 0, mCurrent.length() - 1); + offset = MathUtils.constrain(offset, 0, mCurrent.length()); do { offset = mIterator.preceding(offset); @@ -417,7 +417,7 @@ public class ArrowKeyMovementMethod extends BaseMovementMethod implements Moveme /** {@inheritDoc} */ public int following(int offset) { // always round cursor index into valid string index - offset = MathUtils.constrain(offset, 0, mCurrent.length() - 1); + offset = MathUtils.constrain(offset, 0, mCurrent.length()); do { offset = mIterator.following(offset); |