RFR: 7455: Add support for jolokia JMX service connection [v6]

[Marcus Hirt]

On Mon, 20 Dec 2021 12:27:42 GMT, Martin Skarsaune <duke at openjdk.java.net> wrote:

>> application/org.openjdk.jmc.jolokia/src/main/java/org/openjdk/jmc/jolokia/AbstractCachedDescriptorProvider.java line 45:
>> 
>>> 43: 
>>> 44: /**
>>> 45:  * I cache a list if identified JVMs that can be refreshed in the background by some means of
>> 
>> "I" personal pronomen. "if" identified. 
>> 
>> Better to keep it impersonal and describe what the class does. E.g. "Keeps a list of identified JVMs..."
>> "The {@code AbstractCachedDescriptorProvider} keeps a list of identified JVMs..." would be even better.
>
> I see. Will adjust �� . FYI: I'm on the old Smalltalk extreme object oriented lingo. You may have seen it in SWT and JFace code as well. A bit out of date. 
> I'm currently on 2 week holiday with family, will get back to the PR next year ��

Enjoy your vacation! :)

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

PR: https://git.openjdk.java.net/jmc/pull/332