RFR: JDK-8298405: Support Markdown in Documentation Comments [v30]

Pavel Rappo prappo at openjdk.org
Mon Feb 12 17:48:09 UTC 2024


On Fri, 9 Feb 2024 22:17:43 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:

>> Please review a patch to add support for Markdown syntax in documentation comments, as described in the associated JEP.
>> 
>> Notable features:
>> 
>> * support for `///` documentation comments in `JavaTokenizer`
>> * new module `jdk.internal.md` -- a private copy of the `commonmark-java` library
>> * updates to `DocCommentParser` to treat `///` comments as Markdown
>> * updates to the standard doclet to render Markdown comments in HTML
>
> Jonathan Gibbons has updated the pull request incrementally with one additional commit since the last revision:
> 
>   refactor recent new test case in TestMarkdown.java

src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/HtmlDocletWriter.java line 1465:

> 1463:                         markdownInput.append('\n');
> 1464:                     }
> 1465:                     markdownInput.append(PLACEHOLDER_BLOCK);

There's quite a lot of overlap with the recently simplified https://github.com/openjdk/jdk/blob/7c6316886d1ae86a663d996dae373c42281622fd/src/jdk.internal.md/share/classes/jdk/internal/markdown/MarkdownTransformer.java#L220-L238

If it's impractical to factor out the common bits, maybe we could at least make the respective parts of HtmlDocletWriter as close as possible to those of MarkdownTransformer.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/16388#discussion_r1486548869


More information about the compiler-dev mailing list