<div dir="ltr">Hi all,<div><br></div><div>Would anyone be motivated in reviewing this fix for the tests?</div><div><br></div><div>Thanks!</div><div>Jc</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Aug 2, 2018 at 11:29 AM JC Beyler <<a href="mailto:jcbeyler@google.com">jcbeyler@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi all,<div><br></div><div>I added the fact that the test requires G1 and that unloading is not disabled via 3 new @requires tags.</div><div><br></div><div>Would someone review the test changes for me?</div><div>Webrev: <a href="http://cr.openjdk.java.net/~jcbeyler/8208242/webrev.00/" target="_blank">http://cr.openjdk.java.net/~jcbeyler/8208242/webrev.00/</a></div><div>Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8208242" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8208242</a><br clear="all"><div><br></div><div dir="ltr" class="m_-4078127393797755002gmail_signature"><div dir="ltr">Thanks for your help,<div>Jc</div><div><br></div><div>Ps: I merged the "add a requires for the unloading"  from <a href="https://bugs.openjdk.java.net/browse/JDK-8208206" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8208206</a> into JDK-8208242 and updated the text so that I don't do another trivial webrev for all these tests (hopefully)</div></div></div></div></div>
</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>