[MPlayer-cvslog] r22487 - trunk/libswscale/swscale.c

[Rich Felker]

On Thu, Mar 08, 2007 at 11:05:55AM +0100, Alex Beregszaszi wrote:
> Hi,
> 
> > Log:
> > Add PIX_FMT_PAL8 to isPacked().
> > patch by "Karl H. Beckers", karl.h.beckers gmx net
> > 
> > 
> > Modified: trunk/libswscale/swscale.c
> > ==============================================================================
> > --- trunk/libswscale/swscale.c	(original)
> > +++ trunk/libswscale/swscale.c	Thu Mar  8 11:02:49 2007
> > @@ -116,7 +116,8 @@ untested special converters
> >  			|| (x)==PIX_FMT_NV12 || (x)==PIX_FMT_NV21\
> >  			|| (x)==PIX_FMT_GRAY16BE || (x)==PIX_FMT_GRAY16LE\
> >  			|| (x)==PIX_FMT_GRAY8 || (x)==PIX_FMT_YUV410P)
> > -#define isPacked(x)    ((x)==PIX_FMT_YUYV422 || (x)==PIX_FMT_UYVY422 ||isRGB(x) || isBGR(x))
> > +#define isPacked(x)    ((x)==PIX_FMT_PAL8 || (x)==PIX_FMT_YUYV422 ||\
> > +                        (x)==PIX_FMT_UYVY422 || isRGB(x) || isBGR(x))
> 
> Why not fix the isRGB/isBGR definitions?

PAL8 isn't intrinsically RGB... The palette is, but not the samples in
the image. They're just indices, and could just as easily be used as
indices into a YUV palette if you converted the palette to YUV..

Rich