[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 <<a href="mailto:david.holmes@oracle.com">david.holmes@oracle.com</a>> \
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'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