On Wed, 5 May 2021 19:22:11 GMT, Daniel Fuchs <[email protected]> wrote:
>> Pavel Rappo has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Fixed more typos
>
> src/jdk.management.agent/share/classes/sun/management/jmxremote/ConnectorBootstrap.java
> line 308:
>
>> 306: public static synchronized JMXConnectorServer initialize() {
>> 307:
>> 308: // Load new management properties
>
> Debatable. I guess what was meant here is:
>
> // Loads a new management Properties instance
>
> Or maybe a better comment text would be:
>
> Load and snapshot management properties
>
> Otherwise LGTM.
I've reverted that particular line.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3885
