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 >>
