[OpenJDK 2D-Dev] RFR : 8200281: Add missing @Override annotations in ImageIO plugins

[Kumar Abhishek]

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.abhishek at oracle.com>
Cc: Philip Race <philip.race at oracle.com>; 2d-dev <2d-dev at 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.abhishek at oracle.com>
>> *Cc:* 2d-dev at 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
>>