diff options
author | 2018-02-22 15:03:05 -0800 | |
---|---|---|
committer | 2018-02-22 15:28:50 -0800 | |
commit | 06537f7a6f0772cbef08bdded933828378b2d32a (patch) | |
tree | e1505a3dea19d6585973d2db0d9dd6988bf0f03d /runtime/native/java_lang_Class.cc | |
parent | a502c7202f56b8f5f2c5de567359fa7f6bab406f (diff) |
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
Diffstat (limited to 'runtime/native/java_lang_Class.cc')
-rw-r--r-- | runtime/native/java_lang_Class.cc | 2 |
1 files changed, 1 insertions, 1 deletions
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()); |