Hi Phil & Brian,
Thanks for your review. FYI, I have updated the webrev to include your suggestions : http://cr.openjdk.java.net/~jdv/8152561/webrev.02/ Regards, Jay From: Philip Race Sent: Wednesday, January 25, 2017 11:01 AM To: Brian Burkhalter Cc: Jayathirth D V; Prasanta Sadhukhan; 2d-dev Subject: Re: [OpenJDK 2D-Dev] [9] RFR JDK-8152561: Is it allowed to have zero value for count in TIFFField.createArrayForType() for the rationals } else if((dataType == TIFFTag.TIFF_RATIONAL I know that someone (ahem Brian ;-)) has a lot of "if(" code in here but please make any new ones "if (" Other than that, +1 -phil. On 1/24/17, 1:08 PM, Brian Burkhalter wrote: Hi Jay, I think line 910 of TIFFField needs to state "and count != 1". Also the copyright year should now be 2017. OK to make these changes without another webrev: +1. Thanks, Brian On Jan 18, 2017, at 1:03 AM, Jayathirth D V <HYPERLINK "mailto:jayathirth....@oracle.com"jayathirth....@oracle.com> wrote: Please find updated webrev for review : HYPERLINK "http://cr.openjdk.java.net/%7Ejdv/8152561/webrev.01/"http://cr.openjdk.java.net/~jdv/8152561/webrev.01/