On Fri, 14 Feb 2025 18:19:17 GMT, Leonid Mesnik <lmes...@openjdk.org> wrote:
>> test/lib/jdk/test/lib/security/XMLUtils.java line 557:
>>
>>> 555: }
>>> 556:
>>> 557: // MISC
>>
>> This is a section break for various groups of methods in this utility class.
>> Before this there have been
>>
>> //////////// CONVERT ////////////
>> //////////// QUERY AND EDIT ////////////
>> //////////// SIGN ////////////
>> //////////// VALIDATE ////////////
>>
>> Can you keep the format?
>
> sure, since such comments are cause warning, i just suppress them.
Oh, they were recognized as Markdown javadoc? I didn't think of that.
It's strange that the other 4 show no warning, and then I noticed that changing
it to
//////////// MISC ////////////
// Adds a new rule to "jdk.xml.dsig.secureValidationPolicy"
can also let the warning go away. The problem is it now treats the `///` line
as javadoc and `//` as nothing. This is definitely not right. Maybe I'll do
some cleanup later.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23525#discussion_r1956622006
