Thanks, Alan. Here's the updated webrev:
http://cr.openjdk.java.net/~prappo/8247212/webrev.01/
> On 9 Jun 2020, at 13:33, Alan Bateman <[email protected]> wrote:
>
> On 09/06/2020 13:15, Pavel Rappo wrote:
>> Hello,
>>
>> Please review the change for https://bugs.openjdk.java.net/browse/JDK-8247212
>>
>> http://cr.openjdk.java.net/~prappo/8247212/webrev.00/
>>
>> The patch adds markup that makes it easier to read documentation in cases
>> where code and prose are mixed. The patch also fixes a few typos.
> ServiceCatalog is not in an exported package so surprised it shows up in a
> javadoc build. In any case, this mostly look okay. Can you fix
> ModuleDescriptor L827/828 as the changes makes the line length a bit
> inconsistent with everything around it.
>
> -Alan.
