[MPlayer-dev-eng] [PATCH] GIF support, release quality?

[Joey Parrish]

On Tue, Jan 28, 2003 at 06:23:11PM +0100, Arpi wrote:
> Ah. Yuo misunderstood the purpose of vf_palette. ok it's a bad name.
> it's actually an BGR8->BGRx colorspace converter. If tehre is a palette, it
> will be used, if no palette then grayscale is assumed.
> So it bevaes the same way as scale or lavc or other csp converters.

Yes, except that the format is still BGR8.  lavc and scale can be loaded
based on requested format or format incompatibilities.  How should
mpcodecs_config_vo() be notified of the need to load vf_palette?
vf_palette checks for a palette by checking for the number of planes in
the mpi it gets, but mpi has not been built when mpcodecs_config_vo() is
called.  I've considered adding a new format to codecs.conf for 8bit w/
pal, but this is far worse than adding a new decoder flag.  Presumably,
the decoder will know if the video being decoded needs vf_palette to be
loaded.  So why not add a flag and make it the job of the decoder to
signal the rest of mplayer to load that filter?

--Joey