RFR: 8344197: SM cleanup in java.util.concurrent [v2]

[Alan Bateman]

On Fri, 15 Nov 2024 15:47:43 GMT, Roger Riggs <rriggs at openjdk.org> 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