Hi Evgeny,
I will sponsor this change. Thank you, Daniil From: serviceability-dev <[email protected]> on behalf of Daniil Titov <[email protected]> Date: Tuesday, September 24, 2019 at 9:49 PM To: Evgeny Mandrikov <[email protected]>, David Holmes <[email protected]> Cc: <[email protected]> Subject: Re: JDK-8199136: Dead code in src/jdk.jcmd/share/classes/sun/tools/common/ProcessArgumentMatcher.java Hi Evgeny, The change looks good to me. Thanks! --Daniil From: serviceability-dev <[email protected]> on behalf of Evgeny Mandrikov <[email protected]> Date: Tuesday, September 24, 2019 at 1:32 AM To: David Holmes <[email protected]> Cc: <[email protected]> Subject: Re: RFR: JDK-8199136: Dead code in src/jdk.jcmd/share/classes/sun/tools/common/ProcessArgumentMatcher.java On Tue, Sep 24, 2019 at 8:15 AM David Holmes <[email protected]> wrote: I just spotted this review request languishing in the mailing list. The patch will need to be rebased against latest jdk sources, so please do that. I'll see if I can drum up a second reviewer and get a sponsor. Hi David, No pb, here is rebased patch - http://cr.openjdk.java.net/~godin/8199136/webrev.01/ Regards, Evgeny
