On Mon, 20 Dec 2021 17:33:58 GMT, Alan Bateman <al...@openjdk.org> wrote:

>> Harold Seigel has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   fix test
>
> test/hotspot/jtreg/runtime/BootClassAppendProp/libGetBootClassPathAppendProp.c
>  line 53:
> 
>> 51:   if (err != JVMTI_ERROR_NONE) {
>> 52:     return (*env)->NewStringUTF(env, "wrong error code");
>> 53:   }
> 
> For diagnosability it may be helpful to have this native method throw an 
> exception with the JVMTI error code or alternatively have the return String 
> include the error code rather than a generic message "wrong error code".

Thanks Alan!  I updated the test to include the JVM TI error code in the above 
'wrong error code' message.

-------------

PR: https://git.openjdk.java.net/jdk/pull/6868

Reply via email to