On Thu, 1 Oct 2020 15:02:01 GMT, Vladimir Kempik <vkem...@openjdk.org> wrote:
> Please review this change for hotspot and one test. > There is few JVMTI callback/event functions in jdk which signature doesn't > match specification. > for example: > static jvmtiError JNICALL IsClassUnloadingEnabled(const jvmtiEnv* env, > jboolean* enabled, ...) > but according to jvmti specs it should be: > static jvmtiError JNICALL IsClassUnloadingEnabled(const jvmtiEnv* env, ...) > same with ClassUnload(jvmtiEnv* jvmti_env, JNIEnv* jni_env, const char* name, > ...) in tests > for many years that didn't matter but with coming JEP-391 it becomes > important to make it match the spec > https://developer.apple.com/documentation/apple_silicon/addressing_architectural_differences_in_your_macos_code > This commit makes the above mentioned functions to have signature matching > jvmti specification This pull request has now been integrated. Changeset: c7f00640 Author: Vladimir Kempik <vkem...@openjdk.org> URL: https://git.openjdk.java.net/jdk/commit/c7f00640 Stats: 20 lines in 3 files changed: 17 ins; 0 del; 3 mod 8253899: Make IsClassUnloadingEnabled signature match specification Reviewed-by: sspitsyn, dholmes ------------- PR: https://git.openjdk.java.net/jdk/pull/466