From 9a2a11e65bd32d82754b3fcff74e9e3795327628 Mon Sep 17 00:00:00 2001 From: Jean Chalard Date: Sun, 19 Feb 2012 18:17:35 -0800 Subject: Rewrite a condition Change-Id: I805c9579788d99b015feb2075ad4771756c7adc5 --- java/src/com/android/inputmethod/latin/LatinIME.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'java/src') diff --git a/java/src/com/android/inputmethod/latin/LatinIME.java b/java/src/com/android/inputmethod/latin/LatinIME.java index f284f0d65..3abe514b0 100644 --- a/java/src/com/android/inputmethod/latin/LatinIME.java +++ b/java/src/com/android/inputmethod/latin/LatinIME.java @@ -887,15 +887,15 @@ public class LatinIME extends InputMethodServiceCompatWrapper implements Keyboar // the second one - the first call successfully avoids this test, but the second one // enters. For the moment we rely on noComposingSpan to further reduce the impact. + // TODO: the following is probably better done in resetEntireInputState(). + // it should only happen when the cursor moved, and the very purpose of the + // test below is to narrow down whether this happened or not. Likewise with + // the call to postUpdateShiftState. // We set this to NONE because after a cursor move, we don't want the space // state-related special processing to kick in. mSpaceState = SPACE_STATE_NONE; - if (((mWordComposer.isComposingWord()) - || mVoiceProxy.isVoiceInputHighlighted()) - && (selectionChanged || noComposingSpan)) { - resetEntireInputState(); - } else if (!mWordComposer.isComposingWord()) { + if ((!mWordComposer.isComposingWord()) || selectionChanged || noComposingSpan)) { resetEntireInputState(); } -- cgit v1.2.3-83-g751a