Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 43b1b3ff authored by Dimitry Ivanov's avatar Dimitry Ivanov
Browse files

Fix coretests: Remove dependency on libnativehelper

This dependency was never needed and it makes test app
depend on private platform libraries - which is no longer allowed.

This change fixes these problems by removing dependency and renaming
JNI method.

Test: Install the test and start it.
Change-Id: I9878627148917c1b5928b98f3c86f6e6097093d6
parent efd7b9f5
Loading
Loading
Loading
Loading
+1 −2
Original line number Diff line number Diff line
@@ -19,8 +19,7 @@ include $(CLEAR_VARS)
LOCAL_SRC_FILES := \
    com_android_frameworks_coretests_JNITest.cpp

LOCAL_SHARED_LIBRARIES := \
    libnativehelper
LOCAL_SDK_VERSION := 16

LOCAL_CFLAGS += -Wall -Werror

+3 −21
Original line number Diff line number Diff line
@@ -14,41 +14,23 @@
 * limitations under the License.
 */

#include "nativehelper/JNIHelp.h"
#include <jni.h>

namespace android {

static jint checkFunction(JNIEnv*, jclass) {
extern "C" JNIEXPORT
jint JNICALL Java_com_android_frameworks_coretests_JNITests_checkFunction(JNIEnv*, jclass) {
    return 1;
}

static const JNINativeMethod sMethods[] = {
    /* name, signature, funcPtr */
    { "checkFunction", "()I", (void*) checkFunction },
};

int register_com_android_frameworks_coretests_JNITests(JNIEnv* env) {
    return jniRegisterNativeMethods(env, "com/android/frameworks/coretests/JNITests", sMethods,
            NELEM(sMethods));
}

}

/*
 * JNI Initialization
 */
jint JNI_OnLoad(JavaVM *jvm, void */* reserved */) {
    JNIEnv *e;
    int status;

    // Check JNI version
    if (jvm->GetEnv((void **) &e, JNI_VERSION_1_6)) {
        return JNI_ERR;
    }

    if ((status = android::register_com_android_frameworks_coretests_JNITests(e)) < 0) {
        return JNI_ERR;
    }

    return JNI_VERSION_1_6;
}