On Mon, 24 Nov 2025 01:18:41 GMT, ExE Boss <[email protected]> wrote: >> Steve Armstrong has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Replace legacy ReflectUtil with modern Reflection API >> >> Replaced sun.reflect.misc.ReflectUtil with jdk.internal.reflect.Reflection >> in atomic field updaters. Added getFieldWithAccess() method and >> FieldAndModifiers record to FieldUpdaterUtil to consolidate field >> access checking logic. > > This can also get rid of the last remaining uses of the legacy > `sun.reflect.misc.ReflectUtil`: > > -------------------------------------------------------------------------------- > > (`jdk.internal.reflect.Reflection` is already imported for > `Reflection.getCallerClass()`)
Thanks @ExE-Boss for the review! I've updated the code to replace the legacy `sun.reflect.misc.ReflectUtil` with the modern `jdk.internal.reflect.Reflection` API. The changes consolidate the field access checking into `FieldUpdaterUtil.getFieldWithAccess()` and use a record for `FieldAndModifiers`. ------------- PR Comment: https://git.openjdk.org/jdk/pull/28464#issuecomment-3573569169
