[OpenJDK 2D-Dev] <AWT Dev> [8] Review request for JDK-8025684 - Fix Raw and unchecked warnings java.awt.image classes

srikalyan chandrashekar srikalyan.chandrashekar at oracle.com
Wed Oct 9 18:37:26 UTC 2013


Hi Artem, i have done some changes (more redundant casts identified and
removed), please get the webrev from the same location again . Thanks
for your review.

---
Thanks
kalyan

On 10/2/2013 3:02 AM, Artem Ananiev wrote:
>
> java.awt.image is one of the Java2D packages, so I'm adding 2d-dev to
> CC. Please, wait for at least one approval from Java2D team.
>
> For easier review, I put the webrev here:
>
> http://cr.openjdk.java.net/~art/srikalyc/8025684.00/
>
> It looks fine to me. There is one "unchecked" warning still left, at
> BufferedImage.java:645, it can be fixed by introducing a local
> variable and @SuppressWarnings("unchecked"), but I'm not sure it's
> worth doing.
>
> Thanks,
>
> Artem
>
> On 10/2/2013 1:51 AM, srikalyan chandrashekar wrote:
>> Hi team ,  could someone review the fix
>>      Bug      : https://bugs.openjdk.java.net/browse/JDK-8025684
>>      Webrev   :
>> https://github.com/srikalyc/JDKfixes/blob/master/java.awt.image.raw_unchecked_webrev.zip
>>
>>
>>      Fix      : Raw and unchecked warnings in AWT image classes fixed
>>




More information about the 2d-dev mailing list