[OpenJDK 2D-Dev] [9] RFR JDK-8164931 : Verify if writer.abort() works properly for all writers in IIOWriteProgressListener.

2016-09-16 Thread Jayathirth D V
Hi, Please review the following fix in JDK9 at your convenience: This issue is similar to https://bugs.openjdk.java.net/browse/JDK-4924727 where we made changes to all ImageReader plugins. Bug : https://bugs.openjdk.java.net/browse/JDK-8164931 Webrev : http://cr.openjdk.java.net/~

[OpenJDK 2D-Dev] [9] RFR JDK-8165947: One more page printed before the test page with OpenJDK.

2016-09-16 Thread Prasanta Sadhukhan
Hi All, Please review a fix for a tck failure in jdk9 whereby "banner page" (cover page) is printed by default when print() is called directly without any print dialog being shown. Bug: https://bugs.openjdk.java.net/browse/JDK-8165947 webrev: http://cr.openjdk.java.net/~psadhukhan/8165947/web

Re: [OpenJDK 2D-Dev] [9] RFR JDK-8164931 : Verify if writer.abort() works properly for all writers in IIOWriteProgressListener.

2016-09-16 Thread Sergey Bylokhov
Looks fine, expect the changes in TIFFImageWriter.java. I am not sure I understand why abortRequested was moved from the end of insert(); On 16.09.16 12:04, Jayathirth D V wrote: Hi, Please review the following fix in JDK9 at your convenience: This issue is similar to https://bugs.openjdk

Re: [OpenJDK 2D-Dev] [9] RFR JDK-8164931 : Verify if writer.abort() works properly for all writers in IIOWriteProgressListener.

2016-09-16 Thread Jayathirth D V
Hi Sergey, Thanks for your review. abortRequested in insert() should be at the end of the function only. It has come by mistake while I was creating webrev as I was testing other things. Please find the updated webrev with abortRequested() check at the end of insert() function for review: http:/