RFR: 8367857: Implement JEP 525: Structured Concurrency (Sixth Preview) [v4]
Alan Bateman
alanb at openjdk.org
Thu Oct 23 16:21:14 UTC 2025
On Thu, 23 Oct 2025 14:15:03 GMT, Viktor Klang <vklang at openjdk.org> wrote:
>> Alan Bateman has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 15 additional commits since the last revision:
>>
>> - Sync up from loom repo
>> - Merge branch 'master' into JDK-8367857
>> - Merge branch 'master' into JDK-8367857
>> - Sync up from loom repo
>> - Merge branch 'master' into JDK-8367857
>> - Sync up from loom repo
>> - Merge branch 'master' into JDK-8367857
>> - Merge branch 'master' into JDK-8367857
>> - Improve docs and review feedback
>> - Merge branch 'master' into JDK-8367857
>> - ... and 5 more: https://git.openjdk.org/jdk/compare/9c45d6d9...28617cff
>
> src/java.base/share/classes/java/util/concurrent/StructuredTaskScope.java line 334:
>
>> 332: * #fork forking} of a subtask {@linkplain java.util.concurrent##MemoryVisibility
>> 333: * <i>happen-before</i>} any actions taken by that subtask, which in turn
>> 334: * <i>happen-before</i> the subtask result is {@linkplain Subtask#get() retrieved}.
>
> This may be interpreted as users calling result() so it might be worth clarifying it as:
>
> "<i>happen-before</i> the subtask result is {@linkplain Subtask#get() retrieved} by {@linkplain #join() join()}
The only change here is using the "##" in the link. So this is pre-existing wording (not changed in this update). I agree it could be mis-read and okay to change it.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27392#discussion_r2455861278
More information about the core-libs-dev
mailing list