RFR: 8310892: ScopedValue throwing StructureViolationException should be clearer [v3]

[Jaikiran Pai]

On Mon, 10 Jul 2023 05:05:13 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Docs only update to add a missing `@throws StructureViolationException` and make it clearer when the exception is thrown. In the future we might re-visit this so that the description is in one place rather than in each method.
>
> 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 three additional commits since the last revision:
> 
>  - Merge
>  - s/ ith/with/
>  - Initial commit

src/java.base/share/classes/java/lang/ScopedValue.java line 405:

> 403:          * as a <em>structure violation</em> when the operation completes (normally or with an
> 404:          * exception). In that case, the underlying construct of the {@code StructuredTaskScope}
> 405:          * is closed and {@link StructureViolationException} is thrown.

Hello Alan, does this mean that the original exception (if any) that caused the completion of the operation would be lost and the caller would instead receive just the `StructureViolationException`?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14679#discussion_r1257748611