On Mon, 20 Dec 2021 17:33:58 GMT, Alan Bateman <[email protected]> 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
