commit 9218c5112995969325a8a900924d0cff49824184
author Keisuke Kuroyanagi <ksk@google.com> Thu May 22 16:33:45 2014 +0900
committer Keisuke Kuroyanagi <ksk@google.com> Thu May 22 16:33:45 2014 +0900
tree fefb80c9012e4a6b95b389f7ade8cf0b6f28de7c
parent 5e2c4c0297ddc733265fd324eb44ac5eca4d7228

Fix: NPE in DistracterFilter

Change-Id: I19a92d79cf547aae8f15e7e5fd5309812ae4f2d5

java/src/com/android/inputmethod/latin/utils/DistracterFilter.java

diff --git a/java/src/com/android/inputmethod/latin/utils/DistracterFilter.java b/java/src/com/android/inputmethod/latin/utils/DistracterFilter.java
index 1cf06b4..f1057da 100644
--- a/java/src/com/android/inputmethod/latin/utils/DistracterFilter.java
+++ b/java/src/com/android/inputmethod/latin/utils/DistracterFilter.java
@@ -77,14 +77,16 @@
     public DistracterFilter(final Context context, final List<InputMethodSubtype> enabledSubtypes) {
         mContext = context;
         mLocaleToSubtypeMap = new HashMap<>();
-        for (final InputMethodSubtype subtype : enabledSubtypes) {
-            final Locale locale = SubtypeLocaleUtils.getSubtypeLocale(subtype);
-            if (mLocaleToSubtypeMap.containsKey(locale)) {
-                // Multiple subtypes are enabled for one locale.
-                // TODO: Investigate what we should do for this case.
-                continue;
+        if (enabledSubtypes != null) {
+            for (final InputMethodSubtype subtype : enabledSubtypes) {
+                final Locale locale = SubtypeLocaleUtils.getSubtypeLocale(subtype);
+                if (mLocaleToSubtypeMap.containsKey(locale)) {
+                    // Multiple subtypes are enabled for one locale.
+                    // TODO: Investigate what we should do for this case.
+                    continue;
+                }
+                mLocaleToSubtypeMap.put(locale, subtype);
             }
-            mLocaleToSubtypeMap.put(locale, subtype);
         }
         mLocaleToKeyboardMap = new HashMap<>();
         mSuggest = new Suggest();