RFR: 8353835: Implement JEP 500: Prepare to Make Final Mean Final [v2]
David Holmes
dholmes at openjdk.org
Tue Sep 23 22:04:33 UTC 2025
On Tue, 23 Sep 2025 15:48:43 GMT, Alan Bateman <alanb at openjdk.org> wrote:
>> src/hotspot/share/prims/jni.cpp line 1873:
>>
>>> 1871: if (log_is_enabled(Debug, jni)) {
>>> 1872: fieldDescriptor fd;
>>> 1873: bool found = InstanceKlass::cast(klass)->find_field_from_offset(offset, true, &fd);
>>
>> If you are assuming/expecting an `InstanceKlass` then that should be the type of the parameter. (The existing code needs a cleanup in this area too but that is outside the scope of this PR.)
>
> Okay, but it will mean a cast in the jni_SetXXField callers, I don't think we avoid that before more changes to the existing code.
Yes there has to be a cast somewhere - but only instance classes can have fields so this is a top-level thing.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25115#discussion_r2373549329
More information about the core-libs-dev
mailing list