RFR: 8345079: Simplify/cleanup Exception handling in RMIConnectionImpl [v2]

Kevin Walls kevinw at openjdk.org
Mon Jan 20 20:21:51 UTC 2025


On Mon, 20 Jan 2025 19:12:53 GMT, Daniel Fuchs <dfuchs at openjdk.org> 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


More information about the serviceability-dev mailing list