* commit '4591869d562baef393b6ff63e90378401183dbc7': [HW6] Split processing and applying the event.
This commit is contained in:
commit
caf539079d
3 changed files with 29 additions and 9 deletions
|
@ -81,11 +81,11 @@ public class CombinerChain {
|
|||
}
|
||||
|
||||
/**
|
||||
* Pass a new event through the whole chain.
|
||||
* Process an event through the combining chain, and return a processed event to apply.
|
||||
* @param previousEvents the list of previous events in this composition
|
||||
* @param newEvent the new event to process
|
||||
*/
|
||||
public void processEvent(final ArrayList<Event> previousEvents, final Event newEvent) {
|
||||
public Event processEvent(final ArrayList<Event> previousEvents, final Event newEvent) {
|
||||
final ArrayList<Event> modifiablePreviousEvents = new ArrayList<>(previousEvents);
|
||||
Event event = newEvent;
|
||||
for (final Combiner combiner : mCombiners) {
|
||||
|
@ -97,6 +97,14 @@ public class CombinerChain {
|
|||
break;
|
||||
}
|
||||
}
|
||||
return event;
|
||||
}
|
||||
|
||||
/**
|
||||
* Apply a processed event.
|
||||
* @param event the event to be applied
|
||||
*/
|
||||
public void applyProcessedEvent(final Event event) {
|
||||
if (null != event) {
|
||||
// TODO: figure out the generic way of doing this
|
||||
if (Constants.CODE_DELETE == event.mKeyCode) {
|
||||
|
|
|
@ -175,20 +175,30 @@ public final class WordComposer {
|
|||
}
|
||||
|
||||
/**
|
||||
* Process an input event.
|
||||
* Process an event and return an event, and return a processed event to apply.
|
||||
* @param event the unprocessed event.
|
||||
* @return the processed event.
|
||||
*/
|
||||
public Event processEvent(final Event event) {
|
||||
final Event processedEvent = mCombinerChain.processEvent(mEvents, event);
|
||||
mEvents.add(event);
|
||||
return processedEvent;
|
||||
}
|
||||
|
||||
/**
|
||||
* Apply a processed input event.
|
||||
*
|
||||
* All input events should be supported, including software/hardware events, characters as well
|
||||
* as deletions, multiple inputs and gestures.
|
||||
*
|
||||
* @param event the event to process.
|
||||
* @param event the event to apply.
|
||||
*/
|
||||
public void processEvent(final Event event) {
|
||||
public void applyProcessedEvent(final Event event) {
|
||||
final int primaryCode = event.mCodePoint;
|
||||
final int keyX = event.mX;
|
||||
final int keyY = event.mY;
|
||||
final int newIndex = size();
|
||||
mCombinerChain.processEvent(mEvents, event);
|
||||
mEvents.add(event);
|
||||
mCombinerChain.applyProcessedEvent(event);
|
||||
refreshTypedWordCache();
|
||||
mCursorPositionWithinWord = mCodePointSize;
|
||||
// We may have deleted the last one.
|
||||
|
|
|
@ -762,7 +762,8 @@ public final class InputLogic {
|
|||
resetComposingState(false /* alsoResetLastComposedWord */);
|
||||
}
|
||||
if (isComposingWord) {
|
||||
mWordComposer.processEvent(inputTransaction.mEvent);
|
||||
final Event processedEvent = mWordComposer.processEvent(inputTransaction.mEvent);
|
||||
mWordComposer.applyProcessedEvent(processedEvent);
|
||||
// If it's the first letter, make note of auto-caps state
|
||||
if (mWordComposer.isSingleLetter()) {
|
||||
mWordComposer.setCapitalizedModeAtStartComposingTime(inputTransaction.mShiftState);
|
||||
|
@ -933,7 +934,8 @@ public final class InputLogic {
|
|||
mDictionaryFacilitator.removeWordFromPersonalizedDicts(rejectedSuggestion);
|
||||
}
|
||||
} else {
|
||||
mWordComposer.processEvent(inputTransaction.mEvent);
|
||||
final Event processedEvent = mWordComposer.processEvent(inputTransaction.mEvent);
|
||||
mWordComposer.applyProcessedEvent(processedEvent);
|
||||
}
|
||||
if (mWordComposer.isComposingWord()) {
|
||||
mConnection.setComposingText(getTextWithUnderline(mWordComposer.getTypedWord()), 1);
|
||||
|
|
Loading…
Reference in a new issue