[Mplayer-cvslog] CVS: main/postproc yuv2rgb.c,1.21,1.22

[Michael Niedermayer]

Hi

On Saturday 27 December 2003 21:36, Alex Beregszaszi wrote:
> Hi,
>
> > > @@ -642,8 +642,8 @@
> > >
> > >  int yuv2rgb_c_init_tables (SwsContext *c, const int inv_table[4],
> > >  int
> > > fullRange, int brightness, int contrast, int saturation) {
> > > -    const int bpp= c->dstFormat&0xFF;
> > > -    const int isRgb= (c->dstFormat>>24) != 'R';
> > > +    const int isRgb = IMGFMT_IS_RGB(c->dstFormat);
> > > +    const int bpp =
> > > isRgb?IMGFMT_RGB_DEPTH(c->dstFormat):IMGFMT_BGR_DEPTH(c->dstFormat);
> > > int i;
> >
> > did u test this?
>
> It compiled :) Anyways, you're using IMGFMT_ in the whole code, so it
> should be safe. Btw, see a mail regarding this.
yes IMGFMT is ok but it flips the rgb flag relative to what it was and should 
be i think, IIRC someone allready sended this patch to me privately sometime 
ago and i tested it (it messed the colors up for the c code)

-- 
Michael
level[i]= get_vlc(); i+=get_vlc();		(violates patent EP0266049)
median(mv[y-1][x], mv[y][x-1], mv[y+1][x+1]);	(violates patent #5,905,535)
buf[i]= qp - buf[i-1];				(violates patent #?)
for more examples, see http://mplayerhq.hu/~michael/patent.html
stop it, see http://petition.eurolinux.org & http://petition.ffii.org/eubsa/en