[OpenJDK 2D-Dev] Review Request for JDK-7059970 : Test case: javax/imageio/plugins/png/ITXtTest.java is not closing a file
Jayathirth D V
jayathirth.d.v at oracle.com
Tue Jul 12 12:03:35 UTC 2016
Hi Brian,
That's very good thing to do as it will remove redundant f.delete().
I have updated the webrev for review :
http://cr.openjdk.java.net/~jdv/7059970/webrev.02/
Thanks,
Jay
From: Brian Burkhalter
Sent: Friday, July 08, 2016 11:35 PM
To: Jayathirth D V
Cc: Philip Race; 2d-dev
Subject: Re: [OpenJDK 2D-Dev] Review Request for JDK-7059970 : Test case: javax/imageio/plugins/png/ITXtTest.java is not closing a file
Hi Jay,
Sorry to be picky here but in doTest() could you not instead have
try {
writeTo(file, src);
ITXtTest dst = readFrom(file);
if (dst == null || !dst.equals(src)) {
throw new RuntimeException("Test failed.");
}
} catch (RuntimeException re) {
throw re;
} finally {
file.delete();
}
System.out.println("Test passed.");
and therefore remove f.delete() from writeTo() and readFrom()?
Thanks,
Brian
On Jul 8, 2016, at 12:04 AM, Jayathirth D V <HYPERLINK "mailto:jayathirth.d.v at oracle.com"jayathirth.d.v at oracle.com> wrote:
I can't perform f.delete() in finally block of writeTo() and readFrom() because "test.png" is shared resource between the methods. So I am deleting "test.png" at places where we are throwing RumtimeException.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20160712/b761e53f/attachment.html>
More information about the 2d-dev
mailing list