RFR: 8277090 : jsr166 refresh for jdk19 [v2]

Paul Sandoz psandoz at openjdk.java.net
Tue May 3 20:23:07 UTC 2022


On Mon, 2 May 2022 13:33:33 GMT, Doug Lea <dl at openjdk.org> wrote:

>> This is the jsr166 refresh for jdk19. See https://bugs.openjdk.java.net/browse/JDK-8285450 and https://bugs.openjdk.java.net/browse/JDK-8277090
>
> Doug Lea has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Address review comments

Looks good, although I cannot say i reviewed the fork join pool code as thoroughly, since it is particularly difficult, so i was most looking for consistency in the pattern of code.

src/java.base/share/classes/java/util/concurrent/ForkJoinWorkerThread.java line 76:

> 74:     ForkJoinWorkerThread(ThreadGroup group, ForkJoinPool pool,
> 75:                          boolean useSystemClassLoader,
> 76:                          boolean clearThreadLocals) {

It's tempting to toggle the sense of last boolean argument to be `preserveThreadLocals` for consistency (given the multiple toggles as the value is propagated from the newly added protected constructor), but which means toggling the value at the use sites that you may not wish to change.

-------------

Marked as reviewed by psandoz (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/8490


More information about the core-libs-dev mailing list