On Sat, 12 Apr 2025 05:47:06 GMT, Kim Barrett <kbarr...@openjdk.org> wrote:
>> Brent Christian has updated the pull request incrementally with two
>> additional commits since the last revision:
>>
>> - Make *the method* static, also
>> - Make Method static
>
> test/lib/jdk/test/whitebox/WhiteBox.java line 609:
>
>> 607: throw e;
>> 608: } catch (Throwable t) {
>> 609: throw new RuntimeException(t);
>
> The only checked exceptions that are thrown by the try-body are
> `InvocationTargetException` and
> `IllegalAccessException` (both from `Method::invoke`). The way
> `InvocationTargetException` is being
> handled here seems fine. But I think it would be better to explicitly handle
> `IllegalAccessException` and
> transform it into a `RuntimeException`, and let everything else just
> propagate out untouched, e.g. no
> handlers for `RuntimeException` or `Throwable`.
Thinking about this some more, if we're going to extract and (re)throw
InterruptedException directly, then I prefer my earlier version:
https://github.com/openjdk/jdk/pull/24527#pullrequestreview-2755384786
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24527#discussion_r2041377443
