RFR: 8367862: debug.cpp: Do not print help message for methods ifdef'd out [v2]

Manuel Hässig mhaessig at openjdk.org
Mon Sep 22 07:19:00 UTC 2025


On Fri, 19 Sep 2025 10:37:18 GMT, Kerem Kat <krk at openjdk.org> wrote:

>> The `help` command in `debug.cpp` was out of date, listing only a fraction of the available functions. This and other commands callable from gdb via `call help()`.
>> 
>> I added all commands to the help message, except `pns2` as it is documented as not being useful when called from gdb.
>> 
>> Also fixed is the message for the conditional `pns` command appearing in PRODUCT builds.
>
> Kerem Kat has updated the pull request incrementally with one additional commit since the last revision:
> 
>   address comments

Thank you for addressing my comments, @krk. You missed one "s" (see below), otherwise this looks good to me.

src/hotspot/share/utilities/debug.cpp line 687:

> 685:   tty->print_cr("method metadata.");
> 686:   tty->print_cr("  blob(CodeBlob* p)      - print CodeBlob");
> 687:   tty->print_cr("  dump_vtable(address p) - dumps vtable of the Klass");

Suggestion:

  tty->print_cr("  dump_vtable(address p) - dump vtable of the Klass");

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

Marked as reviewed by mhaessig (Committer).

PR Review: https://git.openjdk.org/jdk/pull/27341#pullrequestreview-3250904144
PR Review Comment: https://git.openjdk.org/jdk/pull/27341#discussion_r2366937608


More information about the hotspot-dev mailing list