Commit 01d47b80 authored Nov 09, 2012 by Jean Chalard

Fix a logic flaw.

This test is seriously wrong : although we do have a method that
does exactly the right thing, this looks at a very specific, unrelated
variable state in hope it happens to be set only in the relevant
cases by the play of logic.
I don't think this changes anything concretely, as the assumption
was probably true in the practice ; still, this code is very hard to
read and prone to unexpected breakage.

Change-Id: I77ab576f90762429fcc752df31ed30e1c81871dc

parent 710d06ce

java/src/com/android/inputmethod/latin/LatinIME.java

+1 −2

Original line number	Diff line number	Diff line
		@@ -1619,8 +1619,7 @@ public final class LatinIME extends InputMethodService implements KeyboardAction
		if (mWordComposer.isComposingWord()) {
		final int length = mWordComposer.size();
		if (length > 0) {
		// Immediately after a batch input.
		if (SPACE_STATE_PHANTOM == spaceState) {
		if (mWordComposer.isBatchMode()) {
		mWordComposer.reset();
		} else {
		mWordComposer.deleteLast();