[OpenJDK 2D-Dev]  request for review 4649812: GIFImageReader handles transparency incorrectly
philip.race at oracle.com
Tue Nov 27 19:29:39 UTC 2012
Looks good to me too. Please mark the bug "noreg-perf".
On 10/3/2012 6:00 AM, Andrew Brygin wrote:
> Hello Vadim,
> the fix looks fine to me.
> On 03.10.2012 16:05, Vadim Pakhnushev wrote:
>> 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
>> But ImageTypeSpecifier.Grayscale, ImageTypeSpecifier.Indexed and
>> ImageTypeSpecifier.Packed specifiers, although retains a copy of
>> passed parameters, don't use them.
More information about the 2d-dev