<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
Ooh, how embarrassing. <br>
Thanks Staffan for adding the correct link.<br>
/F<br>
<br>
<div class="moz-cite-prefix">On 2013-10-10 13:38, Staffan Larsen
wrote:<br>
</div>
<blockquote
cite="mid:918A0955-6236-4916-BD54-F344EC9AADEA@oracle.com"
type="cite">
<meta http-equiv="Content-Type" content="text/html;
charset=ISO-8859-1">
Clicking on the link in this email takes me to the wrong webrev.
The correct URL is in the text: <a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Eallwin/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
moz-do-not-send="true"
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 moz-do-not-send="true"
href="https://bugs.openjdk.java.net/browse/JDK-8024423">https://bugs.openjdk.java.net/browse/JDK-8025638</a><br>
Webrev: <a moz-do-not-send="true"
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 moz-do-not-send="true"
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>
</blockquote>
<br>
</body>
</html>