[FFmpeg-devel] [Ffmpeg-devel] [PATCH] avoid calling a NULL function in swscale.c

Michael Niedermayer michaelni
Tue May 1 11:51:47 CEST 2007


Hi

On Tue, May 01, 2007 at 11:05:04AM +0200, Benoit Fouet wrote:
> Hi Diego,
> 
> Diego Biurrun wrote:
> > On Mon, Apr 30, 2007 at 05:15:49PM +0200, Benoit Fouet wrote:
> >   
> >> Benoit Fouet wrote:
> >>     
> >>> Benoit Fouet wrote:
> >>>   
> >>>       
> >>>> ok, i cannot apply it by myself
> >>>> i attach updated patches (and indentation is done with tabs, to be
> >>>> compliant with what exists in swscale.c)
> >>>>     
> >>>>         
> >>> it'll wait until Diego has finished the cleanup :)
> >>>   
> >>>       
> >> updated patches attached
> >>
> >> --- libswscale/swscale.c	2007-04-30 17:12:35.000000000 +0200
> >> +++ libswscale/swscale.c.1	2007-04-30 17:12:25.000000000 +0200
> >> @@ -1611,6 +1611,8 @@
> >>  
> >> +    if(conv)
> >> +    {
> >>      if (dstStride[0]*srcBpp == srcStride[0]*dstBpp)
> >>          conv(src[0], dst[0] + dstStride[0]*srcSliceY, srcSliceH*srcStride[0]);
> >> @@ -1626,6 +1628,7 @@
> >>      }
> >> +    }
> >>      return srcSliceH;
> >>     
> >
> > Don't forget to fix the indentation in a subsequent commit.
> >
> >   
> the second patch is there to do so :)
> and as i previously told, i can't commit it by myself (i have no commit
> rights in swscale repository)

hmm, are you sure?

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If you really think that XML is the answer, then you definitly missunderstood
the question -- Attila Kinali
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070501/40c3d29b/attachment.pgp>



More information about the ffmpeg-devel mailing list