[FFmpeg-devel] [PATCH] Fix return value for sws_setColorspaceDetails()

[Michael Niedermayer]

On Thu, Apr 16, 2009 at 06:05:29PM +0200, Stefano Sabatini wrote:
> On date Tuesday 2009-04-14 11:57:14 +0200, Stefano Sabatini encoded:
> > Hi,
> > I'm still in the process of figuring out how the whole thing works,
> > but the fix looks obvious.
> > 
> > Regards.
> > -- 
> > FFmpeg = Fostering & Friendly Mind-dumbing Purposeless Ecumenical Geisha
> 
> > Index: ffmpeg/libswscale/swscale.c
> > ===================================================================
> > --- ffmpeg.orig/libswscale/swscale.c	2009-04-14 11:54:53.000000000 +0200
> > +++ ffmpeg/libswscale/swscale.c	2009-04-14 11:54:57.000000000 +0200
> > @@ -2299,7 +2299,7 @@
> >      c->saturation= saturation;
> >      c->srcRange  = srcRange;
> >      c->dstRange  = dstRange;
> > -    if (isYUV(c->dstFormat) || isGray(c->dstFormat)) return 0;
> > +    if (isYUV(c->dstFormat) || isGray(c->dstFormat)) return -1;
> >  
> >      c->uOffset=   0x0400040004000400LL;
> >      c->vOffset=   0x0400040004000400LL;
> 
> Ping?

what about fixing the case instead of failing?

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090416/2fabdaef/attachment.pgp>