[OpenJDK 2D-Dev] [9] Review Request: 8177628 Opensource unit/regression tests for ImageIO
Prahalad Kumar Narayanan
prahalad.kumar.narayanan at oracle.com
Mon May 22 11:23:09 UTC 2017
Hello Sergey
The patch looks good to me.
. I imported the patch and executed automated jtreg tests.
. No failures were noticed on win64 build.
One subtle observation
. One of the test cases- DestTypeTest.java, seems to require VancouverIsland200.jpg.
. Since the file is looked up in a try ... catch block, the test will continue to execute despite image not being a part of the patch.
. You could check this test code & copyrights for the image just to ensure the intended functionality is tested.
Thanks
Have a good day
Prahalad N.
-----Original Message-----
From: Sergey Bylokhov
Sent: Saturday, May 20, 2017 7:51 AM
To: Jayathirth D V; Philip R Race
Cc: 2d-Dev
Subject: [OpenJDK 2D-Dev] [9] Review Request: 8177628 Opensource unit/regression tests for ImageIO
Hello,
Please review the fix for jdk9.
This fix is a part of the effort to opensource the tests in jdk.
In the fix a number of tests for ImageIO are moved to the open ws.
Note that most of the code moved as-is, and probably can be improved in the future, but I tried to limit the number of changes in this move.
Bug: https://bugs.openjdk.java.net/browse/JDK-8177628
Webrev can be found at: http://cr.openjdk.java.net/~serb/8177628/webrev.00
The same fix for JavaSound: http://mail.openjdk.java.net/pipermail/sound-dev/2016-October/000472.html
More information about the 2d-dev
mailing list