<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 24, 2019 at 1:56 PM Sean Mullan <<a href="mailto:sean.mullan@oracle.com">sean.mullan@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 5/23/19 8:14 PM, Arthur Eubanks wrote:<br>
> Ping on a review from security-dev.<br>
> <br>
> On Fri, May 17, 2019 at 9:53 AM Chris Hegarty <<a href="mailto:chris.hegarty@oracle.com" target="_blank">chris.hegarty@oracle.com</a> <br>
> <mailto:<a href="mailto:chris.hegarty@oracle.com" target="_blank">chris.hegarty@oracle.com</a>>> wrote:<br>
> <br>
>     Arthur,<br>
> <br>
>>     On 17 May 2019, at 17:50, Arthur Eubanks <<a href="mailto:aeubanks@google.com" target="_blank">aeubanks@google.com</a><br>
>>     <mailto:<a href="mailto:aeubanks@google.com" target="_blank">aeubanks@google.com</a>>> wrote:<br>
>><br>
>>         Looks good.<br>
>><br>
>>         Trivially, maybe amend the comment to be more explicit<br>
>><br>
>>            86       // SOCKS V4 ( requires IPv4 )<br>
>><br>
>>         -Chris.<br>
>><br>
>>     Done<br>
>>     <a href="http://cr.openjdk.java.net/~aeubanks/8224081/webrev.02/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~aeubanks/8224081/webrev.02/</a><br>
>><br>
>>     I will wait for another review from security-dev.<br>
> <br>
>     You have my Review ( conditional on a Reviewer for the test in the<br>
>     security area ).<br>
<br>
It seems ok but given that this area is a bit unpredictable I would <br>
recommend you be available/online to monitor CI results after you push <br>
the fix in case something breaks.<br></blockquote><div>I will submit next week then. Thanks for the review. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
--Sean<br>
</blockquote></div></div>