On Wed, 25 Mar 2026 21:49:11 GMT, Doug Lea <[email protected]> wrote:
>> Changes signal filtering to avoid possible starvation
>
> Doug Lea has updated the pull request with a new target base due to a merge
> or a rebase. The pull request now contains 80 commits:
>
> - Merge branch 'openjdk:master' into JDK-8373118
> - Merge branch 'openjdk:master' into JDK-8373118
> - Fix DelayScheduler quiescent shutdown; misc tweaks
> - Merge remote-tracking branch 'refs/remotes/origin/JDK-8373118' into
> JDK-8373118
> - Merge branch 'openjdk:master' into JDK-8373118
> - Merge branch 'openjdk:master' into JDK-8373118
> - Initial version of delegated lock redesign
> - reenable submissionQueue bypass
> - Improve performance under oversubscription, step 1
> - Reduce reliance on noUserHelp
> - ... and 70 more: https://git.openjdk.org/jdk/compare/88bdbb78...f2842dcd
src/java.base/share/classes/java/util/concurrent/DelayScheduler.java line 313:
> 311: ScheduledForkJoinTask<?>[] h = heapArray;
> 312: int n = heapSize, cap;
> 313: if (p == null || h == null || n < 0 || (cap = h.length) <= 0)
Looks a bit more consistent to do the following:
Suggestion:
int n, cap;
if (p == null || h == null || (n = heapSize) < 0 || (cap = h.length) <=
0)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28797#discussion_r2994233218
