[OpenJDK 2D-Dev]  Review Request: 8177628 Opensource unit/regression tests for ImageIO
philip.race at oracle.com
Mon May 22 22:22:12 UTC 2017
On 05/22/2017 02:46 PM, Sergey Bylokhov wrote:
> Hi, Prahalad.
> Thank you for review.
> Yes, this image(if found) is used to initialize the BImage, but the test can generates BImage on its own.
> In the new version the usage of ".jpg" is removed:
> I have checked that the test fails on jdk5 where related bug was not fixed.
> ----- prahalad.kumar.narayanan at oracle.com wrote:
>> 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
>> . 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.
>> 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]  Review Request: 8177628 Opensource
>> unit/regression tests for ImageIO
>> 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
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8177628
>> Webrev can be found at:
>> The same fix for JavaSound:
More information about the 2d-dev