<div dir="ltr">Thanks both!<div><br></div><div>Here is the new webrev if someone could push it :-):</div><div><a href="http://cr.openjdk.java.net/~jcbeyler/8208249/webrev.01/">http://cr.openjdk.java.net/~jcbeyler/8208249/webrev.01/</a><br></div><div><br></div><div>Thanks again!</div><div>Jc</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jul 31, 2018 at 9:51 AM Igor Ignatyev <<a href="mailto:igor.ignatyev@oracle.com">igor.ignatyev@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi JC,<br>
<br>
looks good to me.<br>
<br>
-- Igor<br>
<br>
> On Jul 30, 2018, at 4:33 AM, Thomas Schatzl <<a href="mailto:thomas.schatzl@oracle.com" target="_blank">thomas.schatzl@oracle.com</a>> wrote:<br>
> <br>
> Hi,<br>
> <br>
> On Thu, 2018-07-26 at 13:04 -0700, JC Beyler wrote:<br>
>> Hi all,<br>
>> <br>
>> I'm not sure this is the right list, let me know if not.<br>
>> <br>
>> Could someone review this small webrev that puts the<br>
>> GeneratedClassProducer in a ThreadLocal holder to remove the data<br>
>> race on it:<br>
>> <br>
>> Webrev: <a href="http://cr.openjdk.java.net/~jcbeyler/8208249/webrev.00/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~jcbeyler/8208249/webrev.00/</a><br>
>> Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8208249" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8208249</a><br>
>> <br>
>> Thanks,<br>
>> Jc<br>
> <br>
>  looks good to me.<br>
> <br>
> Thomas<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><br></div>Thanks,<div>Jc</div></div></div>