[FFmpeg-devel] [PATCH] lavf/rawutils: Make ff_reshuffle_raw_rgb() somewhat more future-proof

[Reimar Döffinger]

On Sat, Feb 27, 2016 at 02:58:39PM +0100, Mats Peterson wrote:
> And yes, QuickTime has a "default palette" for each bit depth that it can
> use, and no palette will be stored in the video sample description in that
> case. But that's only sensible to use for 1 bpp black & white or 8 bpp
> grayscale (and that's what I'm doing in movenc.c), since in other cases
> comparing each color of the current palette with the default Macintosh
> palette is a bit of overkill in my book.

It's 256 comparisons to save 1 kB.
It's not a big deal, but it seems like fairly cheap optimization to do.