On Fri, 14 Feb 2025 19:23:54 GMT, Weijun Wang <wei...@openjdk.org> wrote:

>> 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.

That said, this is just a test utility class. I'm OK with whatever style at the 
moment. Thanks.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23525#discussion_r1956623296

Reply via email to