[MPlayer-dev-eng] [PATCH] palette

[Joey Parrish]

On Tue, Nov 18, 2003 at 04:02:45PM -0500, D Richard Felker III wrote:
> > This patch is for vf_palette.  The problem is that vf_palette
> > uses conversion routines that assume that the palette itself
> > is the same depth as the output colorspace.  So if it outputs
> > to 16 bit space, then it thinks the pal is 16-bits.  I don't
> > know where this would be true, but all my palette'd 8-bit avi's
> > and gif's and fli's all have 24-bit palette.  Since I'm not
> > about to rewrite the rgb2rgb code, I instead comment out support
> > for BGR/RGB 15/16 in vf_palette.  Now they will output correct
> > images to 24 or 32 bit colorspace, and if the vo needs 15 or 16,
> > vf_scale will do it's job and convert correctly.
> > 
> > Comments?  Is this patch okay?
> 
> IMO it's slow. Why not write a fast converter for each bpp? You should
> also support yuv outputs! :)

I don't have time to write it.  Until I do have time, I'd much rather
have the filter work correctly.

I'm not making it slower than it already was, I'm forcing it to work
correctly 100% of the time instead of 50%.

--Joey

-- 
"Of the seven dwarves, the only one who shaved was Dopey.
That should tell us something about the wisdom of shaving."