RFR: 8344197: SM cleanup in java.util.concurrent
Roger Riggs
rriggs at openjdk.org
Fri Nov 15 15:50:57 UTC 2024
On Fri, 15 Nov 2024 13:30:34 GMT, Doug Lea <dl at openjdk.org> wrote:
>> src/java.base/share/classes/java/util/concurrent/ForkJoinPool.java line 1108:
>>
>>> 1106: implements ForkJoinWorkerThreadFactory {
>>> 1107: public final ForkJoinWorkerThread newThread(ForkJoinPool pool) {
>>> 1108: return new ForkJoinWorkerThread(null, pool, true, false);
>>
>> This doesn't look right. For the common pool it should return InnocuousForkJoinWorkerThread(pool).
>
> I agree on both counts with Alan: The FJP changes are not quite right, and in any case should be postponed until after current FJP PR, and then further simplified, which would be easiest if I did this.
Revert all changes.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22119#discussion_r1844086214
More information about the core-libs-dev
mailing list