On Mon, 17 Apr 2023 21:29:53 GMT, Jonathan Gibbons <j...@openjdk.org> wrote:
>> Please review a cleanup in DocCommentParser to merge blockContent and >> inlineContent into a single method to parse "rich content" in a doc comment. >> >> **Note:** This is dependent on PR #13362, to convert `DocCommentParser` to >> use enhanced switch. > > Jonathan Gibbons has updated the pull request with a new target base due to a > merge or a rebase. The pull request now contains ten commits: > > - Merge remote-tracking branch 'upstream/master' into 8305713.dcp-content > - Merge > - Merge branch 'pr/13362' into pr/13362 > - convert additional switch statements > - JDK-8305673: Convert DocCommentParser to use enhanced switch > - avoid conflict with dependent PR > - Merge remote-tracking branch 'upstream/master' into 8305713.dcp-content > - Merge remote-tracking branch 'upstream/master' into 8305713.dcp-content > - JDK-8305713: DocCommentParser: merge blockContent and inlineContent Genuine question: what benefits are there in merging the blockContent method with the inlineContent method? (See initial, trivial comments inline.) src/jdk.compiler/share/classes/com/sun/tools/javac/parser/DocCommentParser.java line 170: > 168: > 169: /** > 170: * Reads "rich text" content, consisting of text, html and inline > tags, Nit: HTML src/jdk.compiler/share/classes/com/sun/tools/javac/parser/DocCommentParser.java line 183: > 181: * <li>{@code BODY}: the beginning of a block tag, or when readung > from > 182: * an HTML file, the appearance of {@code </main>} (or {@code > </body>}, > 183: * as determined by {@link #isEndBody()} Lost trailing ) src/jdk.compiler/share/classes/com/sun/tools/javac/parser/DocCommentParser.java line 190: > 188: * > 189: * > 190: */ Nit: needless blank lines src/jdk.compiler/share/classes/com/sun/tools/javac/parser/DocCommentParser.java line 190: > 188: * > 189: * > 190: */ Shouldn't that be POSTAMBLE? src/jdk.compiler/share/classes/com/sun/tools/javac/parser/DocCommentParser.java line 190: > 188: * > 189: * > 190: */ Typo: readung src/jdk.compiler/share/classes/com/sun/tools/javac/parser/DocCommentParser.java line 838: > 836: * <li>cdata: {@code <![CDATA[ ... ]]>} > 837: * </ul> > 838: * or Dangling "or". ------------- PR Review: https://git.openjdk.org/jdk/pull/13431#pullrequestreview-1390076976 PR Review Comment: https://git.openjdk.org/jdk/pull/13431#discussion_r1170009701 PR Review Comment: https://git.openjdk.org/jdk/pull/13431#discussion_r1170019297 PR Review Comment: https://git.openjdk.org/jdk/pull/13431#discussion_r1170010347 PR Review Comment: https://git.openjdk.org/jdk/pull/13431#discussion_r1170011228 PR Review Comment: https://git.openjdk.org/jdk/pull/13431#discussion_r1170011527 PR Review Comment: https://git.openjdk.org/jdk/pull/13431#discussion_r1170024803