RFR [15] 8247212: Use assistant markup in java.lang.module.ModuleDescriptor

Pavel Rappo pavel.rappo at oracle.com
Tue Jun 9 13:33:28 UTC 2020


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 <Alan.Bateman at oracle.com> 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.



More information about the core-libs-dev mailing list