Ooh, how embarrassing.
Thanks Staffan for adding the correct link.
/F
On 2013-10-10 13:38, Staffan Larsen wrote:
Clicking on the link in this email takes me to the wrong webrev. The
correct URL is in the text:
http://cr.openjdk.java.net/~allwin/farvidss/8025638/webrev.00/
<http://cr.openjdk.java.net/%7Eallwin/farvidss/8025638/webrev.00/>
/Staffan
On 10 okt 2013, at 12:42, Fredrik Arvidsson
<fredrik.arvids...@oracle.com <mailto:fredrik.arvids...@oracle.com>>
wrote:
Hi
Please help me to review the changes below:
Jira case: https://bugs.openjdk.java.net/browse/JDK-8025638
<https://bugs.openjdk.java.net/browse/JDK-8024423>
Webrev:
http://cr.openjdk.java.net/~allwin/farvidss/8025638/webrev.00/
<http://cr.openjdk.java.net/%7Eallwin/farvidss/8024423/webrev.00/>
About this change.
A previous change (https://bugs.openjdk.java.net/browse/JDK-8010278)
in the Tool.java class caused any tool deriving from this base class
return the wrong value to the caller when failing.
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.
Previously failed Aurora tests have been run using UTE and verified
to PASS.
Cheers
/Fredrik
