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