On Wed, 18 Nov 2020 19:53:42 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:

>> Hi @plummercj,
>>      For old jmap, if "gz" option is used in commandline, it prompt usage 
>> info indicating that option is illegal.
>>      if "gz" option is not used, the op->arg(2) will be an empty string "", 
>> which is set from the attacher side in the method execute() of 
>> VirtualMachineImpl.java.
>
> Ok, I see now that 3 args are always written, even if empty, so you at least 
> get a "" if there is no value for the argument position.

I think you need to add a test case. You can use the work done for JDK-8237354 
dcmd as a template. See 
http://cr.openjdk.java.net/~rschmelter/webrevs/8237354/webrev.3/. I don't know 
that's it's necessary to add a test case for each GC, but if you do, I suggest 
putting them all in one file. The use of separate files per test case was not 
necessary.

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

PR: https://git.openjdk.java.net/jdk/pull/1251

Reply via email to