On Mon, 20 Jan 2025 19:12:53 GMT, Daniel Fuchs <[email protected]> wrote:
>> Kevin Walls 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 five additional
>> commits since the last revision:
>>
>> - Remove unnecessary/dup catches
>> - Merge remote-tracking branch 'upstream/master' into
>> 8345079_RMIConnectionImpl_excep
>> - unnecessary catches
>> - (c)
>> - 8345079: Simplify/cleanup Exception handling in RMIConnectionImpl
>
> src/java.management.rmi/share/classes/javax/management/remote/rmi/RMIConnectionImpl.java
> line 1008:
>
>> 1006: throw (RuntimeException) e;
>> 1007: } else if (e instanceof RuntimeException) {
>> 1008: throw (RuntimeException) e;
>
> duplicated? see lines 1005-1006
oops 8-)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23072#discussion_r1922817438
