On Tue, 16 Jun 2026 08:05:03 GMT, Alan Bateman <[email protected]> wrote:

>> src/java.base/share/classes/java/lang/reflect/Field.java line 1499:
>> 
>>> 1497:      * record class or hidden class),
>>> 1498:      */
>>> 1499:     private boolean isFinalInstanceInNormalClass() {
>> 
>> The implementation of this method hasn't been updated in this PR and thus it 
>> now considers the final instance in a value class as belonging to a normal 
>> class. Should this method's implementation (and its comment) be updated to 
>> not consider value classes as normal?
>
> Core reflection is implemented on method handles so you won't see the changes 
> in Field.java. For fields in value classes, the "field accessor" will be 
> generated as read-only so the setter methods always throws.

This is being fixed in:
- https://github.com/openjdk/valhalla/pull/2548

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31123#discussion_r3421343253

Reply via email to