RFR: 8340213: jcmd VM.events ignores max argument [v2]

Kevin Walls kevinw at openjdk.org
Tue Sep 17 17:00:05 UTC 2024


On Mon, 16 Sep 2024 19:54:38 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

>> The inner  'int max;' declaration hide previous max.
>
> Leonid Mesnik has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - typo fixed.
>  - check added.

If this is not too late:
it's odd that max is a STRING parameter, and the command does the conversion itself.  Is it possible to change to an INT parameter?  I can log it for the future if this change is all tested and ready to go.

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

PR Comment: https://git.openjdk.org/jdk/pull/21024#issuecomment-2356448754


More information about the serviceability-dev mailing list