RFR: 8364544: Extract the checks of AtomicXxxFieldUpdater into a common place [v2]

Steve Armstrong duke at openjdk.org
Tue Nov 25 03:45:45 UTC 2025


On Tue, 25 Nov 2025 03:06:09 GMT, Chen Liang <liach at openjdk.org> 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.
>
> Changes requested by liach (Reviewer).

Thanks @liach! Refactored to use `findValidatedField()` as suggested. Much cleaner API - eliminates the redundant record and combines everything into one call.

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

PR Comment: https://git.openjdk.org/jdk/pull/28464#issuecomment-3573661859


More information about the core-libs-dev mailing list