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

Reply via email to