<div dir="ltr"><div dir="ltr"><div>Hi Andrew,</div><div><br></div><div>Thanks for having a look at this.</div><div><br></div><div>Webrev.02 without "All rights reserved" and "affiliates" parts:</div><div><br></div><div dir="ltr"> * <a href="http://cr.openjdk.java.net/~mbalao/webrevs/8213154/8213154.webrev.02/" target="_blank">http://cr.openjdk.java.net/~<wbr>mbalao/webrevs/8213154/<wbr>8213154.webrev.02/</a><br></div><div dir="ltr"> * <a href="http://cr.openjdk.java.net/~mbalao/webrevs/8213154/8213154.webrev.02.zip" target="_blank">http://cr.openjdk.java.net/~<wbr>mbalao/webrevs/8213154/<wbr>8213154.webrev.02.zip</a></div><div dir="ltr"><br></div><div>Are you okay to go?</div><div><br></div><div>Kind regards,</div><div>Martin.-</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 1, 2018 at 8:12 AM, Andrew Hughes <span dir="ltr"><<a href="mailto:gnu.andrew@redhat.com" target="_blank">gnu.andrew@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue, 30 Oct 2018 at 18:00, Martin Balao <<a href="mailto:mbalao@redhat.com">mbalao@redhat.com</a>> wrote:<br>
><br>
> Hi,<br>
><br>
> You're right, this is not relevant for a test.<br>
><br>
>  * <a href="http://cr.openjdk.java.net/~mbalao/webrevs/8213154/8213154.webrev.01" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~<wbr>mbalao/webrevs/8213154/<wbr>8213154.webrev.01</a><br>
>  * <a href="http://cr.openjdk.java.net/~mbalao/webrevs/8213154/8213154.webrev.01.zip" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~<wbr>mbalao/webrevs/8213154/<wbr>8213154.webrev.01.zip</a><br>
><br>
> Thanks,<br>
> Martin.-<br>
><br>
> On Tue, Oct 30, 2018 at 2:50 PM, Alan Bateman <<a href="mailto:Alan.Bateman@oracle.com">Alan.Bateman@oracle.com</a>> wrote:<br>
>><br>
>> On 30/10/2018 17:44, Martin Balao wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> Can I have a review for JDK-8213154 [1]?<br>
>><br>
>>  * <a href="http://cr.openjdk.java.net/~mbalao/webrevs/8213154/8213154.webrev.00/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~<wbr>mbalao/webrevs/8213154/<wbr>8213154.webrev.00/</a><br>
>>  * <a href="http://cr.openjdk.java.net/~mbalao/webrevs/8213154/8213154.webrev.00.zip" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~<wbr>mbalao/webrevs/8213154/<wbr>8213154.webrev.00.zip</a><br>
>><br>
>> Did you mean to include a test in this update? Just asking because tests doesn't usually have the Classpath exception.<br>
>><br>
>> -Alan<br>
><br>
><br>
<br>
</div></div>Classpath exception addition looks fine and appropriate for the JDK<br>
code. The "All rights reserved" additions are unnecessary for RH copyrights.<br>
<span class="HOEnZb"><font color="#888888">-- <br>
Andrew :)<br>
<br>
Senior Free Java Software Engineer<br>
Red Hat, Inc. (<a href="http://www.redhat.com" rel="noreferrer" target="_blank">http://www.redhat.com</a>)<br>
<br>
Web Site: <a href="http://fuseyism.com" rel="noreferrer" target="_blank">http://fuseyism.com</a><br>
Twitter: <a href="https://twitter.com/gnu_andrew_java" rel="noreferrer" target="_blank">https://twitter.com/gnu_<wbr>andrew_java</a><br>
PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://<a href="http://keys.gnupg.net" rel="noreferrer" target="_blank">keys.gnupg.net</a>)<br>
Fingerprint = 5132 579D D154 0ED2 3E04  C5A0 CFDA 0F9B 3596 4222<br>
</font></span></blockquote></div><br></div>