Hi Dmitry, I have updated the year in the copyright section for the modified file. Please find the updated Webrev:- http://cr.openjdk.java.net/~jdv/8200281/webrev.03/
Thanks, Abhishek -----Original Message----- From: Dmitry Markov Sent: Wednesday, July 29, 2020 11:45 PM To: Kumar Abhishek <kumar.z.abhis...@oracle.com> Cc: Alexey Ivanov <alexey.iva...@oracle.com>; Philip Race <philip.r...@oracle.com>; 2d-dev <2d-dev@openjdk.java.net> Subject: Re: [OpenJDK 2D-Dev] RFR : 8200281: Add missing @Override annotations in ImageIO plugins Hi Abhishek, Could you update the year in the copyright section for the modified files, please? Regards, Dmitry > On 29 Jul 2020, at 11:43, Kumar Abhishek <kumar.z.abhis...@oracle.com> wrote: > > Hi Alexey/Dmitry, > > Please find the updated Webrev. > http://cr.openjdk.java.net/~jdv/8200281/webrev.02/ > > I have added the missed annotation. > > Thanks, > Abhishek > -----Original Message----- > From: Alexey Ivanov > Sent: Saturday, July 25, 2020 2:38 AM > To: Kumar Abhishek <kumar.z.abhis...@oracle.com> > Cc: Philip Race <philip.r...@oracle.com>; 2d-dev > <2d-dev@openjdk.java.net> > Subject: Re: [OpenJDK 2D-Dev] RFR : 8200281: Add missing @Override > annotations in ImageIO plugins > > Hi Abhishek, > > *JPEGImageWriter.java* > > 240 public IIOMetadata convertStreamMetadata(IIOMetadata inData, > > 255 public IIOMetadata > 256 convertImageMetadata(IIOMetadata inData, > > 1751 public synchronized void dispose() { > > These methods are missed: no @Override annotation. In the last case, the > method implements an interface rather than overrides a method from a super > class, yet the @Override annotation is usually added. > > *BMPImageReader.java* > 1937 public void imageProgress(ImageReader source, > 1945 public void imageUpdate(ImageReader source, > 1956 public void passComplete(ImageReader source, > > > So the patch is incomplete. > Please verify you add @Override annotation to all overridden methods in these > classes. > > > Regards, > Alexey > > On 24/07/2020 18:59, Philip Race wrote: >> This is fine. Approved. >> >> -phil. >> >> On 7/24/20, 3:08 AM, Kumar Abhishek wrote: >>> >>> Hi Phil, >>> >>> This bug was originally only for JPEG plugin though I have verified >>> the changes for other plugins as well. >>> >>> Do you want me to divide the change for different plugins and create >>> separate bugs or the current approach is fine? >>> >>> Please find the updated webrev with superfluous comments removed : >>> >>> http://cr.openjdk.java.net/~jdv/8200281/webrev.01/ >>> <http://cr.openjdk.java.net/%7Ejdv/8200281/webrev.01/> >>> >>> Thanks, >>> >>> Abhishek >>> >>> *From:*Philip Race >>> *Sent:* Thursday, July 23, 2020 10:37 PM >>> *To:* Kumar Abhishek <kumar.z.abhis...@oracle.com> >>> *Cc:* 2d-dev@openjdk.java.net >>> *Subject:* Re: [OpenJDK 2D-Dev] RFR : 8200281: Add missing @Override >>> annotations in ImageIO plugins >>> >>> >>> >>> On 7/23/20, 5:20 AM, Kumar Abhishek wrote: >>> >>> Hello, Please review this small change for JDK-16 . >>> >>> This is a clean up task to verify and add missing annotations in >>> ImageIO plugins >>> >>> >>> They are missing in part because most of these classes were added in >>> 1.4 and >>> annotations were introduced in 1.5. So I am not sure how much >>> retrofitting we should do along these lines. If applied widely it >>> could just make backports painful when patches will not apply. >>> >>> >>> /** Overrides the method defined in the superclass. */ >>> + @Override >>> >>> The comment is now superflous so I suggest to remove it in the >>> couple of places it is present. >>> >>> -phil. >>> >>> Verified and added annotations for all the Writer and Reader >>> implementation under >>> src/java.desktop/share/classes/com/sun/imageio/plugins/ >>> >>> Bug/webrev : >>> >>> https://bugs.openjdk.java.net/browse/JDK-8200281 >>> >>> http://cr.openjdk.java.net/~jdv/8200281/webrev.00/ >>> <http://cr.openjdk.java.net/%7Ejdv/8200281/webrev.00/> >>> >>> >>> >>> >>> >>> Thanks, Abhishek >>> >
