<div dir="ltr"><div dir="ltr">Thanks Thomas!<div><br></div><div>Here is the new webrev for the curious, which puts the includes in lexicographical order: </div><div><br></div><div>Webrev: <a href="http://cr.openjdk.java.net/~jcbeyler/8212083/webrev.01/">http://cr.openjdk.java.net/~jcbeyler/8212083/webrev.01/</a></div><div>And Bug is still here: <a href="https://bugs.openjdk.java.net/browse/JDK-8212083" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8212083</a></div><div><br></div><div>I'll pass it through the submit repo and then, upon success I'll push it with you and Paul as reviewers.</div><div><br></div><div>Thanks again,</div><div>Jc</div></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 15, 2018 at 1:47 AM Thomas Schatzl <<a href="mailto:thomas.schatzl@oracle.com">thomas.schatzl@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,<br>
<br>
On Thu, 2018-10-11 at 15:23 -0700, JC Beyler wrote:<br>
> Hi all,<br>
> <br>
> I've cleaned up the gc/lock tests to use the ExceptionCheckingJniEnv<br>
> wrapper and now we have all JNI calls in that test group covered from<br>
> a "checking exceptions and returns of JNI methods".<br>
> <br>
> Let me know what you think:<br>
> Webrev: <a href="http://cr.openjdk.java.net/~jcbeyler/8212083/webrev.00/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~jcbeyler/8212083/webrev.00/</a><br>
> Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8212083" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8212083</a><br>
> <br>
<br>
  looks good to me as far as I understand the code. One nit: could you<br>
fix the #include declarations: they should be ordered<br>
lexicographically.<br>
<br>
Thanks,<br>
  Thomas<br>
<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>