RFR: 8340334: Update jcmd VM.events max parameter to be INT [v2]

Chris Plummer cjplummer at openjdk.org
Thu Nov 21 00:06:18 UTC 2024


On Thu, 21 Nov 2024 00:02:32 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

>> The  jcmd VM.events max parameter type is changed to INT.
>> Also,I noted the max <= 0 is ignored, so I updated documentation and set "0" as a default value.
>> The jcmd exists if parameter is negative now.
>> 
>> The `max` is `int` while really it is the unsigned int. However I don't think it makes sense to change it to `size_t` or `unsigned int` in all places where 'max' is used.
>
> Leonid Mesnik has updated the pull request incrementally with one additional commit since the last revision:
> 
>   doc fixed

Marked as reviewed by cjplummer (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/22224#pullrequestreview-2449954991


More information about the serviceability-dev mailing list