On Mon, 9 Jan 2023 10:13:54 GMT, Pavel Rappo <pra...@openjdk.org> wrote:
>> 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.
Agreed that we should not do "cosmetic" changes on imported code.
-------------
PR: https://git.openjdk.org/jdk/pull/11701
