Re: RFR: XXS JDK-8213056,,Nested anchor tags in java.lang.module

2018-10-27 Thread Alan Bateman
On 26/10/2018 23:10, Jonathan Gibbons wrote: Please review a couple of very small changes to fix some HTML issues in the generated docs. The first change removes an empty paragraph; the second avoids causing nested tags. Looks good (I had to read the bug to understand how this nested anchor

Re: RFR: XXS JDK-8213056,,Nested anchor tags in java.lang.module

2018-10-26 Thread Mandy Chung
Looks good. Mandy On 10/26/18 3:10 PM, Jonathan Gibbons wrote: Please review a couple of very small changes to fix some HTML issues in the generated docs. The first change removes an empty paragraph; the second avoids causing nested tags. JBS: https://bugs.openjdk.java.net/browse/JDK-82130

Re: RFR: XXS JDK-8213056,,Nested anchor tags in java.lang.module

2018-10-26 Thread joe darcy
+1 -Joe On 10/26/2018 3:10 PM, Jonathan Gibbons wrote: Please review a couple of very small changes to fix some HTML issues in the generated docs. The first change removes an empty paragraph; the second avoids causing nested tags. JBS: https://bugs.openjdk.java.net/browse/JDK-8213056 --

RFR: XXS JDK-8213056,,Nested anchor tags in java.lang.module

2018-10-26 Thread Jonathan Gibbons
Please review a couple of very small changes to fix some HTML issues in the generated docs. The first change removes an empty paragraph; the second avoids causing nested tags. JBS: https://bugs.openjdk.java.net/browse/JDK-8213056 -- Jon Changes inline here: $ hg diff diff -r 3a767a000aa