commit 14fe931362054affe1bf14cf053589f93d5b7229
author Keisuke Kuroyanagi <ksk@google.com> Wed Feb 19 11:24:39 2014 +0900
committer Keisuke Kuroyanagi <ksk@google.com> Wed Feb 19 11:24:39 2014 +0900
tree 006d580df25b4b980fda4a8737540451bbdb4d10
parent 85fe06e759ab89c629caab5fb73de7ff0441d060

Fix: using wrong method to call ArrayList.add.

bug: 13072808
Change-Id: I60dc5ae1e535f2f0b3784a12c40ac082560a436f

native/jni/src/suggest/core/dictionary/word_property.cpp

diff --git a/native/jni/src/suggest/core/dictionary/word_property.cpp b/native/jni/src/suggest/core/dictionary/word_property.cpp
index 288e6b0..4733118 100644
--- a/native/jni/src/suggest/core/dictionary/word_property.cpp
+++ b/native/jni/src/suggest/core/dictionary/word_property.cpp
@@ -42,7 +42,7 @@
         jintArray bigramWord1CodePointArray = env->NewIntArray(word1CodePoints->size());
         env->SetIntArrayRegion(bigramWord1CodePointArray, 0 /* start */,
                 word1CodePoints->size(), &word1CodePoints->at(0));
-        env->CallVoidMethod(outBigramTargets, addMethodId, bigramWord1CodePointArray);
+        env->CallBooleanMethod(outBigramTargets, addMethodId, bigramWord1CodePointArray);
         env->DeleteLocalRef(bigramWord1CodePointArray);
 
         int bigramProbabilityInfo[] = {bigramProperty->getProbability(),
@@ -51,7 +51,7 @@
         jintArray bigramProbabilityInfoArray = env->NewIntArray(NELEMS(bigramProbabilityInfo));
         env->SetIntArrayRegion(bigramProbabilityInfoArray, 0 /* start */,
                 NELEMS(bigramProbabilityInfo), bigramProbabilityInfo);
-        env->CallVoidMethod(outBigramProbabilities, addMethodId, bigramProbabilityInfoArray);
+        env->CallBooleanMethod(outBigramProbabilities, addMethodId, bigramProbabilityInfoArray);
         env->DeleteLocalRef(bigramProbabilityInfoArray);
     }
 
@@ -62,11 +62,11 @@
         jintArray shortcutTargetCodePointArray = env->NewIntArray(targetCodePoints->size());
         env->SetIntArrayRegion(shortcutTargetCodePointArray, 0 /* start */,
                 targetCodePoints->size(), &targetCodePoints->at(0));
-        env->CallVoidMethod(outShortcutTargets, addMethodId, shortcutTargetCodePointArray);
+        env->CallBooleanMethod(outShortcutTargets, addMethodId, shortcutTargetCodePointArray);
         env->DeleteLocalRef(shortcutTargetCodePointArray);
         jobject integerProbability = env->NewObject(integerClass, intToIntegerConstructorId,
                 mShortcuts[i].getProbability());
-        env->CallVoidMethod(outShortcutProbabilities, addMethodId, integerProbability);
+        env->CallBooleanMethod(outShortcutProbabilities, addMethodId, integerProbability);
         env->DeleteLocalRef(integerProbability);
     }
     env->DeleteLocalRef(integerClass);