+1
--alex
On 09/04/2018 17:10, [email protected] wrote:
Hi Jc,
It looks good to me.
Thank you for this cleanup!
Thanks,
Serguei
On 9/4/18 16:12, JC Beyler wrote:
Hi all,
Continuing the removal of the JNI_ENV* macros, I have done about half
of the JVMTI Get[A-F] methods. The whole change for jvmti/GetXX tests
was about 2k of changes so it seemed best to just divide it into two
parts (from my approximation there will be 2 more webrevs to finish
the umbrella item JDK-8209547).
The change is straightforward as before, just a bit repetitive:
Webrev: http://cr.openjdk.java.net/~jcbeyler/8210198/webrev.00/
<http://cr.openjdk.java.net/%7Ejcbeyler/8210198/webrev.00/>
Bug: https://bugs.openjdk.java.net/browse/JDK-8210198
I tested this in release mode for the tests that were modified.
Thanks,
Jc
