RFR (M) 8210198: Clean up JNI_ENV_ARG for vmTestbase/jvmti/Get[A-F] tests

Alex Menkov alexey.menkov at oracle.com
Wed Sep 5 19:14:16 UTC 2018


+1

--alex

On 09/04/2018 17:10, serguei.spitsyn at oracle.com 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
> 


More information about the serviceability-dev mailing list