On Fri, 17 Jan 2025 18:18:48 GMT, Kevin Walls <kev...@openjdk.org> wrote:
>> src/java.management.rmi/share/classes/javax/management/remote/rmi/RMIConnectionImpl.java
>> line 212:
>>
>>> 210: throw (SecurityException) e;
>>> 211: if (e instanceof RuntimeException)
>>> 212: throw (RuntimeException) e;
>>
>> Throughout all these changes: SecurityException is a RuntimeException, so
>> you do not need to check for both.
>>
>> An alternative is to:
>>
>> } catch (RuntimeException r) {
>> throw r;
>> } catch (Exception e) {
>> ....
>> }
>
> OK sure, I was just making it super explicit. Can update.
Updated - re-ran all of the management tests, looks good.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23072#discussion_r1920640875
