RFR (M) 8210665: Clean up JNI_ENV_ARG and factorize the macros for vmTestbase/jvmti[R-U] tests

JC Beyler jcbeyler at google.com
Wed Sep 12 18:45:09 UTC 2018


Hi all,

I am continuing the clean up of the testbase with the next batch, I know
this is getting repetitive but bear with me please, after this webrev, we
have in vmTestbase:

- 29 more files that have the JNI_ENV* or JVMTI_ENV* macros (for a
subsequent webrev)
- 400+ files that have trivial #ifdef __cplusplus macros around the extern
"C" and the final } (for a second subsequent webrev)

After those two webrev, we can go to doing more important refactoring to
get the vmTestbase in shape to start migrating out of there hopefully.

So, without further ado, here is another one with 50 file changes and 1283
line changes.

Webrev: http://cr.openjdk.java.net/~jcbeyler/8210665/webrev.00/
Bug: https://bugs.openjdk.java.net/browse/JDK-8210665

This passes testing for the changed tests on my dev machine.

Thanks for your reviews,
Jc
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20180912/0ef933bb/attachment.html>


More information about the serviceability-dev mailing list