On Mon, 2 May 2022 13:33:33 GMT, Doug Lea <d...@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