On Tue, 23 Dec 2025 09:47:02 GMT, Kevin Walls <[email protected]> wrote:

> Comment-only cleanup, didn't spot during previous review.
> 
> Trivial, no code affected.

src/hotspot/share/services/diagnosticCommand.cpp line 97:

> 95:   // Registration of the diagnostic commands.
> 96:   // Argument specifies to which interfaces a command is made available,
> 97:   // can also specify if hidden from jcmd help.

I kind a liked the "first argument, second argument" approach a bit more.

// First argument specifies on which interfaces a command is made available.
// Optional second argument specifies if hidden from jcmd help. Default is 
false.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28962#discussion_r2643629971

Reply via email to