<div dir="ltr">Thanks for the quick reviews!<br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 6, 2017 at 9:06 PM, Philip Race <span dir="ltr"><<a href="mailto:philip.race@oracle.com" target="_blank">philip.race@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hmm. I pointed this out in the review of the original fix<br>
<a href="http://mail.openjdk.java.net/pipermail/2d-dev/2016-March/006491.html" rel="noreferrer" target="_blank">http://mail.openjdk.java.net/p<wbr>ipermail/2d-dev/2016-March/006<wbr>491.html</a><br>
and thought it was resolved .. I should have looked more closely at the update.<br>
<br>
The fix looks fine although I also think this should be main/othervm as<br>
this test is de-registering SPIs and in same VM modefor jtreg then subsequent<br>
Image I/O tests may fail. I am actually surprised this hasn't been a problem.<br>
<br>
Perhaps you can include that in this update ?<br></blockquote><div><br></div><div>I'm resisting doing that. I don't know anything about imageio, and I'd like to keep it that way. Probably there should be more investigation from an expert on all the imageio tests. Leaving this to you.</div><div><br></div><div>Speaking of scope creep, we should fix the typo creatImageOutputStream (even if your name is Ken Thompson).</div></div></div></div>