On Fri, 31 Jul 2026 06:21:27 GMT, David Holmes <[email protected]> wrote:

>> Please review this simple change to give a more informative error message 
>> for `UnsupportedClassVersionError` when trying to load a recent classfile on 
>> an older VM. The wording used has already been finalized - see JBS.
>> 
>> Testing:
>> 
>> - Updated test runtime/ClassFile/UnsupportedClassFileVersion.java. 
>> - Added a new test case to 
>> `test/hotspot/jtreg/runtime/ClassFile/PreviewVersion.java` to show preview 
>> classfiles are also handled as expected
>> 
>> Tier 1 sanity testing.
>> 
>> Thanks
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> David Holmes has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Update existing UnsupportedClassFileVersion test with new message.
>   Restore Versions.java as no change needed.

src/hotspot/share/classfile/classFileParser.cpp line 4584:

> 4582:       "to %u.0, but the class file version of %s is %u.%u.",
> 4583:       class_name->as_C_string(), JDK_VERSION, 
> JVM_CLASSFILE_MAJOR_VERSION,
> 4584:       class_name->as_C_string(), major, minor);

That's really helpful, thanks Alex.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32117#discussion_r3690451278

Reply via email to