+1

--alex

On 08/27/2018 15:41, Chris Plummer wrote:
Looks good.

Chris

On 8/27/18 3:37 PM, Igor Ignatyev wrote:
http://cr.openjdk.java.net/~iignatyev//8186548/webrev.00/index.html
10 lines changed: 4 ins; 4 del; 2 mod;
Hi all,

could you please review this small patch which puts JcmdBase into the directory w/ the only two tests which use it?

JBS: https://bugs.openjdk.java.net/browse/JDK-8186548
webrev: http://cr.openjdk.java.net/~iignatyev//8186548/webrev.00/index.html
testing: sun/tools/jcmd tests + jdk-tier[1-3]

Thanks,
-- Igor


Reply via email to