[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: RFR: JDK-8199136: Dead code in src/jdk.jcmd/share/classes/sun/tools/common/ProcessArgumentMatche
From:       Evgeny Mandrikov <mandrikov () gmail ! com>
Date:       2019-09-24 8:30:00
Message-ID: CAEPFu6-WNZAOHPpoeJ-ja2Z9-1eotfnUNZzxBLXQvUrKdMd17g () mail ! gmail ! com
[Download RAW message or body]

On Tue, Sep 24, 2019 at 8:15 AM David Holmes <david.holmes@oracle.com>
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

[Attachment #3 (text/html)]

<div dir="ltr"><div dir="ltr"><div><div dir="ltr" class="gmail_signature" \
data-smartmail="gmail_signature"><div dir="ltr"><div>On Tue, Sep 24, 2019 at 8:15 AM \
David Holmes &lt;<a href="mailto:david.holmes@oracle.com">david.holmes@oracle.com</a>&gt; \
wrote:<br></div></div></div></div></div><div class="gmail_quote"><blockquote \
class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex"> I just spotted this review request languishing in \
the mailing list.<br></blockquote><div>  </div><blockquote class="gmail_quote" \
style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex"> The patch will need to be rebased against latest \
jdk sources, so please <br> do that. I&#39;ll see if I can drum up a second reviewer \
and get a sponsor.<br></blockquote><div><br></div><div>Hi \
David,</div><div><br></div><div>No pb, here is rebased patch -  <a \
href="http://cr.openjdk.java.net/~godin/8199136/webrev.01/">http://cr.openjdk.java.net \
/~godin/8199136/webrev.01/</a></div><div><br></div><div><br></div><div>Regards,</div><div>Evgeny<br></div></div></div>




[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic