On Fri, 15 Nov 2024 15:47:43 GMT, Roger Riggs <[email protected]> wrote:
>> src/java.base/share/classes/java/util/concurrent/atomic/AtomicIntegerFieldUpdater.java
>> line 399:
>>
>>> 397: if ((ccl != null) && (ccl != cl) &&
>>> 398: ((cl == null) || !isAncestor(cl, ccl))) {
>>> 399:
>>> sun.reflect.misc.ReflectUtil.checkPackageAccess(tclass);
>>
>> The ensureMemberAccess needs to stay but the checkPackageAccess can be
>> removed.
>>
>> @seanjmullan I think you'll want to study this one closely.
>
> ok, removing package access check
I see you've removed it from AtomicIntegerFieldUpdater. There are also
checkPackageAccess usages in AtomicLongFieldUpdater and
AtomicReferenceFieldUpdater.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22119#discussion_r1844228641
