Refactor all phantom-space-inserted spaces in one place (0e84041b) · Commits · e / os / android_packages_inputmethods_LatinIME

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

+10 −5

Original line number	Diff line number	Diff line
		@@ -1405,7 +1405,7 @@ public final class LatinIME extends InputMethodService implements KeyboardAction
		mHandler.postUpdateSuggestionStrip();
		final CharSequence text = specificTldProcessingOnTextInput(rawText);
		if (SPACE_STATE_PHANTOM == mSpaceState) {
		sendKeyCodePoint(Keyboard.CODE_SPACE);
		promotePhantomSpace();
		}
		mConnection.commitText(text, 1);
		mConnection.endBatchEdit();
		@@ -1568,7 +1568,7 @@ public final class LatinIME extends InputMethodService implements KeyboardAction
		mWordComposer.setBatchInputWord(batchInputText);
		mConnection.beginBatchEdit();
		if (SPACE_STATE_PHANTOM == mSpaceState) {
		sendKeyCodePoint(Keyboard.CODE_SPACE);
		promotePhantomSpace();
		}
		mConnection.setComposingText(batchInputText, 1);
		mExpectingUpdateSelection = true;
		@@ -1724,7 +1724,7 @@ public final class LatinIME extends InputMethodService implements KeyboardAction
		// Sanity check
		throw new RuntimeException("Should not be composing here");
		}
		sendKeyCodePoint(Keyboard.CODE_SPACE);
		promotePhantomSpace();
		}

		// NOTE: isCursorTouchingWord() is a blocking IPC call, so it often takes several
		@@ -1802,7 +1802,7 @@ public final class LatinIME extends InputMethodService implements KeyboardAction

		if (SPACE_STATE_PHANTOM == spaceState &&
		mCurrentSettings.isPhantomSpacePromotingSymbol(primaryCode)) {
		sendKeyCodePoint(Keyboard.CODE_SPACE);
		promotePhantomSpace();
		}
		sendKeyCodePoint(primaryCode);

		@@ -2071,7 +2071,7 @@ public final class LatinIME extends InputMethodService implements KeyboardAction
		int firstChar = Character.codePointAt(suggestion, 0);
		if ((!mCurrentSettings.isWeakSpaceStripper(firstChar))
		&& (!mCurrentSettings.isWeakSpaceSwapper(firstChar))) {
		sendKeyCodePoint(Keyboard.CODE_SPACE);
		promotePhantomSpace();
		}
		}

		@@ -2249,6 +2249,11 @@ public final class LatinIME extends InputMethodService implements KeyboardAction
		mHandler.postUpdateSuggestionStrip();
		}

		// This essentially inserts a space, and that's it.
		public void promotePhantomSpace() {
		sendKeyCodePoint(Keyboard.CODE_SPACE);
		}

		// Used by the RingCharBuffer
		public boolean isWordSeparator(final int code) {
		return mCurrentSettings.isWordSeparator(code);