On Fri, 15 Nov 2024 17:46:07 GMT, Roger Riggs <rri...@openjdk.org> wrote:
>> Refactored to remove use of doPrivileged() and use of SecurityManager. >> The DefaultForkJoinWorkerThreadFactory no longer uses the SM to target a >> common thread pool. >> >> A careful review is requested. > > Roger Riggs has updated the pull request incrementally with one additional > commit since the last revision: > > Remove additional package access checks in AtomicLongFieldUpdater and > AtomicReferenceFieldUpdater These now look OK to me too. A side question: Are there now any situations in which Thread.interrupt can throw an exception? If not, a few more things in j.u.c and elsewhere could be simplified someday. ------------- Marked as reviewed by dl (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/22119#pullrequestreview-2439532591
