diff options
author | 2014-07-29 15:53:12 +0000 | |
---|---|---|
committer | 2014-07-29 15:53:12 +0000 | |
commit | f5705385360e7f660b3f899480396a06a1dc6079 (patch) | |
tree | 753101495312b2b83bbc92cf5fd99fd8564dc070 /java/src/com/android/inputmethod/event/CombinerChain.java | |
parent | e4f81d6b4700a54fddd0bca27926ebc65591a427 (diff) | |
parent | c9bba853fa00f03bfc9b49931e320957adc15b20 (diff) | |
download | latinime-f5705385360e7f660b3f899480396a06a1dc6079.tar.gz latinime-f5705385360e7f660b3f899480396a06a1dc6079.tar.xz latinime-f5705385360e7f660b3f899480396a06a1dc6079.zip |
am c9bba853: am 666f1b38: Merge "Revert "[HW7.5] Introduce the @Nonnull annotation"" into lmp-dev
* commit 'c9bba853fa00f03bfc9b49931e320957adc15b20':
Revert "[HW7.5] Introduce the @Nonnull annotation"
Diffstat (limited to 'java/src/com/android/inputmethod/event/CombinerChain.java')
-rw-r--r-- | java/src/com/android/inputmethod/event/CombinerChain.java | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/java/src/com/android/inputmethod/event/CombinerChain.java b/java/src/com/android/inputmethod/event/CombinerChain.java index f69bf4fd0..8ddc9e7d9 100644 --- a/java/src/com/android/inputmethod/event/CombinerChain.java +++ b/java/src/com/android/inputmethod/event/CombinerChain.java @@ -24,8 +24,6 @@ import com.android.inputmethod.latin.Constants; import java.util.ArrayList; import java.util.HashMap; -import javax.annotation.Nonnull; - /** * This class implements the logic chain between receiving events and generating code points. * @@ -89,7 +87,6 @@ public class CombinerChain { * @return the processed event. It may be the same event, or a consumed event, or a completely * new event. However it may never be null. */ - @Nonnull public Event processEvent(final ArrayList<Event> previousEvents, final Event newEvent) { final ArrayList<Event> modifiablePreviousEvents = new ArrayList<>(previousEvents); Event event = newEvent; @@ -97,6 +94,10 @@ public class CombinerChain { // A combiner can never return more than one event; it can return several // code points, but they should be encapsulated within one event. event = combiner.processEvent(modifiablePreviousEvents, event); + if (null == event) { + // Combiners return null if they eat the event. + break; + } } return event; } |