RFR: JDK-8289106: Add model of class file versions to core reflection [v6]
Joe Darcy
darcy at openjdk.org
Tue Aug 16 18:24:23 UTC 2022
On Tue, 16 Aug 2022 14:40:41 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> Joe Darcy has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 18 additional commits since the last revision:
>>
>> - Implement code review feedback.
>> - Update ClassFileVersionTest.java.
>> - Merge branch 'master' into JDK-8289106
>> - Appease jcheck.
>> - Finish more precise versioned location support; udpate tests.
>> - Partial implementation of version-depdendent locations() with tests.
>> - Merge branch 'master' into JDK-8289106
>> - Merge branch 'master' into JDK-8289106
>> - Add method to map from major class file version.
>> - Merge branch 'master' into JDK-8289106
>> - ... and 8 more: https://git.openjdk.org/jdk/compare/9da21735...2e6dfc8e
>
> src/java.base/share/classes/java/lang/reflect/ClassFileFormatVersion.java line 259:
>
>> 257: RELEASE_20(64);
>> 258:
>> 259: private int major;
>
> Could be final?
Sure; that is a better expression of the intent of the field. Will change before pushing; thanks.
-------------
PR: https://git.openjdk.org/jdk/pull/9299
More information about the core-libs-dev
mailing list