On Mon, 29 Sep 2025 18:26:40 GMT, Volkan Yazici <[email protected]> wrote:

>> Alan Bateman has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   RemoveFields(duration) and filter internal frames
>
> test/jdk/java/lang/reflect/Field/mutateFinals/jni/JNIAttachMutator.java line 
> 59:
> 
>> 57:             this.value = value;
>> 58:         }
>> 59:     }
> 
> Do we need to extend this list with a copy of itself where `value` is static?

This test is for the access check in Field.set when there is no Java caller. 
It's only applicable to final instance fields (can't mutate a static final with 
this API).

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25115#discussion_r2390644652

Reply via email to