Re: RFR (L) 8210429: Clean up JNI_ENV_ARG for vmTestbase/jvmti/Get[G-Z] tests

Wed, 05 Sep 2018 20:09:02 -0700

Hi JC,

In getstacktr008.cpp, this should be one line:

 240         classDef.class_byte_count =
 241             env->GetArrayLength(classBytes);

Otherwise looks great!

Chris

On 9/5/18 6:39 PM, JC Beyler wrote:
Hi all,

Continuing the removal of the JNI_ENV* macros, I have done the other half of the JVMTI Get[G-Z] methods. The final JVMTI test refactoring will come in a final webrev after this one.

The change is straightforward as before, just a bit repetitive:

I tested this in release mode for the tests that were modified.

Thanks for your reviews,
Jc


Reply via email to