RFR: 8335610: DiagnosticFramework: CmdLine::is_executable() correction
Daniel D. Daugherty
dcubed at openjdk.org
Fri Jul 12 23:20:55 UTC 2024
On Wed, 3 Jul 2024 13:58:51 GMT, Kevin Walls <kevinw at openjdk.org> wrote:
> CmdLine::is_executable() looks wrong, surely an empty line is not executable.
>
> With this change, in DCmd::parse_and_execute() we will avoid needlessly entering the code block to try and execute the command.
>
> DCmd tests all good:
> make images test TEST="test/hotspot/jtreg/serviceability/dcmd test/jdk/sun/tools/jcmd"
Is it useful to allow (but ignore) an "empty" command to perhaps allow
perf analysis of the dcmd mechanism?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/20006#issuecomment-2226515899
More information about the hotspot-runtime-dev
mailing list