RFR: 8358701: Remove misleading javax.management.remote API doc wording about JMX spec, and historic link to JMXMP [v3]
Kevin Walls
kevinw at openjdk.org
Sat Jun 7 16:59:52 UTC 2025
On Fri, 6 Jun 2025 19:02:53 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:
>> Kevin Walls has updated the pull request incrementally with one additional commit since the last revision:
>>
>> update
>
> src/java.management/share/classes/javax/management/remote/package-info.java line 41:
>
>> 39: *
>> 40: * <p>A connector makes an MBean server remotely accessible through
>> 41: * a given protocol.
>
> Nit: Before, this line was ended with semicolon sign `:` as there is the list following it.
> Now: The list follows the dot `.`. Should the dot be replaced with a semicolon?
> (It impacts the CSR if you decide to change it.)
Thanks Serguei. The colon made sense before, the text was clearly introducing a list of facts.
With the change, I think it looks good with or without. Happy to leave as is, or update if there is any objection.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25670#discussion_r2133997955
More information about the serviceability-dev
mailing list