Hi Alex and Chris, thanks for your reviews.
-- Igor > On Aug 27, 2018, at 4:16 PM, Alex Menkov <alexey.men...@oracle.com> wrote: > > +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
