RFR (M) 8210182: Remove macros for C compilation from vmTestBase but non jvmti
Alex Menkov
alexey.menkov at oracle.com
Thu Aug 30 17:47:59 UTC 2018
Hi Jc,
It looks much better now.
BTW did you run all the tests?
test/hotspot/jtreg/vmTestbase/nsk/share/jni/JNIreferences.cpp
"(V" should be "()V":
// notify another thread that JNI local reference has been created
- JNI_ENV_PTR(env)->CallVoidMethod(JNI_ENV_ARG_3(env,
createWicket, JNI_ENV_PTR(env)->GetMethodID(JNI_ENV_ARG_4(env, klass,
"unlock", "()V"))));
+ env->CallVoidMethod(createWicket, env->GetMethodID(klass,
"unlock", "(V"));
// wait till JNI local reference can be released (it will
heppen then we will leave the method)
- JNI_ENV_PTR(env)->CallVoidMethod(JNI_ENV_ARG_3(env,
deleteWicket, JNI_ENV_PTR(env)->GetMethodID(JNI_ENV_ARG_4(env, klass,
"waitFor", "()V"))));
+ env->CallVoidMethod(deleteWicket, env->GetMethodID(klass,
"waitFor", "(V"));
}
--alex
On 08/29/2018 22:01, JC Beyler wrote:
> Hi all,
>
> A follow-up to Igor's work on getting tests in C++, I am working on
> simplifying the macros in the tests from the vmTestBase. The full change
> being a bit too large, I'm cutting it up in pieces to be easier to
> review and integrate.
>
> Here is the first part, it changes all vmTestbase tests outside the
> vmTestbase/jvmti subfolder:
>
> Webrev: http://cr.openjdk.java.net/~jcbeyler/8210182/webrev.01/
> <http://cr.openjdk.java.net/%7Ejcbeyler/8210182/webrev.01/>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8210182
>
> Thanks!
> Jc
More information about the serviceability-dev
mailing list