RFR: 8346050: Update BuildTestLib.gmk to build whole testlibrary [v4]

Weijun Wang weijun at openjdk.org
Fri Feb 14 19:27:12 UTC 2025


On Fri, 14 Feb 2025 19:23:54 GMT, Weijun Wang <weijun at 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


More information about the build-dev mailing list