<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Clicking on the link in this email takes me to the wrong webrev. The correct URL is in the text: <a href="http://cr.openjdk.java.net/~allwin/farvidss/8025638/webrev.00/">http://cr.openjdk.java.net/~allwin/farvidss/8025638/webrev.00/</a><div><br></div><div>/Staffan</div><div><br><div><div>On 10 okt 2013, at 12:42, Fredrik Arvidsson <<a href="mailto:fredrik.arvidsson@oracle.com">fredrik.arvidsson@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">
<meta http-equiv="content-type" content="text/html; charset=ISO-8859-1">
<div text="#000000" bgcolor="#FFFFFF">
Hi<br>
<br>
Please help me to review the changes below:<br>
<br>
Jira case: <a href="https://bugs.openjdk.java.net/browse/JDK-8024423">https://bugs.openjdk.java.net/browse/JDK-8025638</a><br>
Webrev: <a href="http://cr.openjdk.java.net/%7Eallwin/farvidss/8024423/webrev.00/">http://cr.openjdk.java.net/~allwin/farvidss/8025638/webrev.00/</a><br>
<br>
About this change.<br>
A previous change (<a href="https://bugs.openjdk.java.net/browse/JDK-8010278">https://bugs.openjdk.java.net/browse/JDK-8010278</a>)
in the Tool.java class caused any tool deriving from this base class
return the wrong value to the caller when failing. <br>
Changes were made to the Tool.java class and to the derived tool
implementation classes to handle errors/exceptions during execution
and ensure that the tool returns 1 to the caller if it fails, and 0
if it succeeds. <br>
<br>
Previously failed Aurora tests have been run using UTE and verified
to PASS.<br>
<br>
Cheers<br>
/Fredrik
</div>
</blockquote></div><br></div></body></html>