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

Reply via email to