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