[OpenJDK 2D-Dev] [8] request for review 4649812: GIFImageReader handles transparency incorrectly

Vadim Pakhnushev vadim.pakhnushev at oracle.com
Wed Oct 3 12:05:24 UTC 2012


Hello,

Please review the fix for http://bugs.sun.com/view_bug.do?bug_id=4649812
webrev: http://cr.openjdk.java.net/~bae/4649812/webrev.00/

This will result in lower footprint actually (not that it will be 
noticeable in real application though).
First, we will omit allocating temporary alpha LUT.
Second, although IndexColorModel will allocate full rgba array in any 
case, we will skip creating ImageTypeSpecifier.Indexed class (created 
with ImageTypeSpecifier.createIndexed call) which store a copy of passed 
LUT arrays.
It seems that this Indexed class was intended to have equals and 
hashCode methods like ImageTypeSpecifier.Banded and 
ImageTypeSpecifier.Interleaved.
But ImageTypeSpecifier.Grayscale, ImageTypeSpecifier.Indexed and 
ImageTypeSpecifier.Packed specifiers, although retains a copy of passed 
parameters, don't use them.

Thanks,
Vadim



More information about the 2d-dev mailing list