On Mon, 11 Mar 2024 17:10:34 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:

>> Kevin Walls has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   (C) oops
>
> src/java.management.rmi/share/classes/javax/management/remote/rmi/RMIConnector.java
>  line 618:
> 
>> 616:                 throw new SecurityException("Subject Delegation has 
>> been removed.");
>> 617:             }
>> 618:         }
> 
> Can't this constructor be removed? It's a private inner class so we should be 
> able to throw before calling that constructor?

Yes that can disappear with no impact.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18025#discussion_r1521324287

Reply via email to