On Fri, 6 Jun 2025 19:02:53 GMT, Serguei Spitsyn <sspit...@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

Reply via email to