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 Thanks for dropping the FJP changes from this patch. I did another pass over the updates in revision 9632ac50 and it looks okay. I've set the Reviewer count to 2 so that Doug or Viktor get a chance to review before this is integrated. ------------- Marked as reviewed by alanb (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/22119#pullrequestreview-2439307715