Looks good! You could also get rid of the <blockquote> which is non-standard for code samples. I would have preserved the 2-space indent in the code sample, but yeah, the java standard is 4-space.
On Tue, Aug 25, 2015 at 12:40 AM, Ivan Gerasimov <ivan.gerasi...@oracle.com> wrote: > Thank you Martin! > > On 24.08.2015 23:49, Martin Buchholz wrote: > > Thanks for doing this. > > The code sample in SwitchPoint.java should be reformatted for consistency > with other javadoc - insert " * " before every line. > > Done. > > Did you check for & as well? > > No &, but grep found one in java.lang.Character.java. > I wasn't sure what is the way to specify it in {@code}, so just replaced > it with a single space. > The resulting html looks Okay. > > The code below needs another {@code as previously in the same javadoc. > > + * The name {@code <init>} will be parsed into { '<', "init", '>'}} > > > Done. > > > On Mon, Aug 24, 2015 at 1:30 PM, Ivan Gerasimov <ivan.gerasi...@oracle.com > > wrote: > >> Hi! >> >> In a few places {@code } block contains < and > sequences, which >> are displayed verbatim in the doc. >> Would you please help review the trivial cleanup? >> >> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8134356 >> WEBREV: http://cr.openjdk.java.net/~igerasim/8134356/00/webrev/ >> >> Sincerely yours, >> Ivan >> > > >