Hello Prahalad, This review is with respect to [1]. Picky comments:
PNGImageReader: 479, 570, 667: Add space around the “-“ signs. 741, 752, 757 and in other files: In general, comments starting with “/*” should be blank after the “/*” with the actual verbiage starting on the next line. What about the PNG ImageWriter? The description in the issue [2] implies that the the ImageCreationTime should be used to set the Creation Time in the text chunk(s). Or is this already happening? Thanks, Brian [1] http://cr.openjdk.java.net/~pnarayanan/8164971/webrev.02/ [2] https://bugs.openjdk.java.net/browse/JDK-8164971 On Jul 20, 2017, at 7:27 AM, Prahalad Kumar Narayanan <prahalad.kumar.naraya...@oracle.com> wrote: > First, Thanks to Brian and Jay for their time in review & feedback. > . I 've addressed the review suggestions and the updated code is available > for review. > . Webrev Link: http://cr.openjdk.java.net/~pnarayanan/8164971/webrev.01/
