On 19/05/2016 05:52, David Holmes wrote:
Not sure who really owns this file so cc'ing core-libs and serviceability.

bug: https://bugs.openjdk.java.net/browse/JDK-8157188

The file src/java.base/share/native/include/classfile_constants.h describes information about classfiles and is used by libverify and ./demo/share/jvmti/java_crw_demo/java_crw_demo.c

This file has not been updated for classfile version 53 and so asserts will fail in java_crw_demo.c when it encounters classes compiled for version 53 - as they now are. This version change caused test failures in the hotspot forest when it was pulled down earlier this week.

This fix trivially bumps the current version number to 53 to fix the failing tests. It is a separate issue as to whether other changes are needed in this file to reflect what is new with classfile version 53.
It's the serviceability area that has historically maintained this. The change looks fine.

-Alan

Reply via email to