On Fri, 22 Nov 2024 21:50:12 GMT, Alex Menkov <amen...@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
