• Jed Denlea's avatar
    image/gif: try harder to use global color table · 31cd20a7
    Jed Denlea authored
    The GIF format allows for an image to contain a global color table which
    might be used for some or every frame in an animated GIF.  This palette
    contains 24-bit opaque RGB values.  An individual frame may use the
    global palette and enable transparency by picking one number to be
    transparent, instead of the color value in the palette.
    
    image/gif decodes a GIF, which contains an []*image.Paletted that holds
    each frame.  When decoded, if a frame has a transparent color and uses
    the global palette, a copy of the global []color.Color is made, and the
    transparency color index is replaced with color.RGBA{}.
    
    When encoding a GIF, each frame's palette is encoded to the form it
    might exist in a GIF, up to 768 bytes "RGBRGBRGBRGB...". If a frame's
    encoded palette is equal to the encoded global color table, the frame
    will be encoded with the flag set to use the global color table,
    otherwise the frame's palette will be included.
    
    So, if the color in the global color table that matches the transparent
    index of one frame wasn't black (and it frequently is not), reencoding a
    GIF will likely result in a larger file because each frame's palette
    will have to be encoded inline.
    
    This commit takes a frame's transparent color index into account when
    comparing an individual image.Paletted's encoded color table to the
    global color table.
    
    Fixes #22137
    
    Change-Id: I5460021da6e4d7ce19198d5f94a8ce714815bc08
    Reviewed-on: https://go-review.googlesource.com/68313Reviewed-by: default avatarNigel Tao <nigeltao@golang.org>
    31cd20a7
writer.go 11.1 KB