On Mon, 9 Jan 2023 09:46:14 GMT, Andrey Turbanov <aturba...@openjdk.org> wrote:

>> Support for Markdown comments in the standard doclet.
>> 
>> To enable Markdown in a comment, start the comment with `/**md` followed by 
>> whitespace.  The syntax is as defined for CommonMark.
>> 
>> The work is in 3 parts:
>> 
>> 1. Update the Compiler Tree API to support Markdown tree nodes, containing 
>> strings of (uninterpreted) Markdown source code.
>> 2. Import commonmark-java into the `jdk.javadoc` module, to be able to 
>> convert Markdown strings to HTML.
>> 3. Update the standard doclet, to leverage the preceding two parts, to 
>> translate Markdown in documentation comments to `Content` nodes.
>> 
>> There are new tests both for the low level work in the Compiler Tree API, 
>> and for the overall high-level work in the doclet.
>
> src/jdk.javadoc/share/classes/jdk/internal/org/commonmark/internal/Bracket.java
>  line 48:
> 
>> 46:     public boolean bracketAfter = false;
>> 47: 
>> 48:     static public Bracket link(Text node, Position markerPosition, 
>> Position contentPosition, Bracket previous, Delimiter previousDelimiter) {
> 
> Let's use blessed modifiers order
> Suggestion:
> 
>     public static Bracket link(Text node, Position markerPosition, Position 
> contentPosition, Bracket previous, Delimiter previousDelimiter) {

This class belongs to a 3rd party library, CommonMark. Changes that aren't 
necessary for the library to work in the JDK or not related to security might 
not be worth it.

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

PR: https://git.openjdk.org/jdk/pull/11701

Reply via email to