jmx-dev RFR: 8344394: Remove SecurityManager and related calls from java.management.rmi [v2]

[Kevin Walls]

On Fri, 22 Nov 2024 21:50:12 GMT, Alex Menkov <amenkov at openjdk.org> wrote:

>> Kevin Walls has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove last import sun.reflect.misc.ReflectUtil
>
> src/java.management.rmi/share/classes/javax/management/remote/rmi/RMIConnectionImpl.java line 108:
> 
>> 106:         ClassLoaderRepository repository = mbeanServer.getClassLoaderRepository();
>> 107:         this.classLoaderWithRepository = new ClassLoaderWithRepository(repository, dcl);
>> 108:         this.defaultContextClassLoader = new CombinedClassLoader(Thread.currentThread().getContextClassLoader(), dcl);
> 
> `this.` are not needed

Sure, they are the way it already was but happy to take them out. 
"this.mbeanServer =" does not need the "this" either, but I'm not touching that line so it would be odd to remove it now.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22270#discussion_r1856568255