From 06537f7a6f0772cbef08bdded933828378b2d32a Mon Sep 17 00:00:00 2001 From: Igor Murashkin Date: Thu, 22 Feb 2018 15:03:05 -0800 Subject: native: Cleanup jni usage code Make the C++ signature consistent with the JNI descriptor by using the same type as the descriptor whenever possible. e.g. "()Ljava/lang/String;" should actually be "jstring fn(JNIEnv*,jobject)" instead of "jobject fn(JNIEnv*,jobject)". Bug: 35325126 Change-Id: I72318525fc3b18b013b8d6fa604d8dd6b5dd4400 Test: make -j32 test-art-host # and also manually that it boots --- runtime/native/java_lang_Class.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'runtime/native/java_lang_Class.cc') diff --git a/runtime/native/java_lang_Class.cc b/runtime/native/java_lang_Class.cc index e518553292..a4a8a376fb 100644 --- a/runtime/native/java_lang_Class.cc +++ b/runtime/native/java_lang_Class.cc @@ -532,7 +532,7 @@ static jobjectArray Class_getDeclaredConstructorsInternal( } static jobject Class_getDeclaredMethodInternal(JNIEnv* env, jobject javaThis, - jobject name, jobjectArray args) { + jstring name, jobjectArray args) { ScopedFastNativeObjectAccess soa(env); DCHECK_EQ(Runtime::Current()->GetClassLinker()->GetImagePointerSize(), kRuntimePointerSize); DCHECK(!Runtime::Current()->IsActiveTransaction()); -- cgit v1.2.3-59-g8ed1b