[Ffmpeg-cvslog] r6162 - trunk/libavcodec/vc1.c

Michael Niedermayer michaelni
Mon Sep 4 21:09:11 CEST 2006


Hi

On Mon, Sep 04, 2006 at 12:04:57PM -0400, Rich Felker wrote:
> On Mon, Sep 04, 2006 at 01:22:04PM +0200, Michael Niedermayer wrote:
> > [...]
> > >      /* skip B-frames if we don't have reference frames */
> > > -    if(s->last_picture_ptr==NULL && (s->pict_type==B_TYPE || s->dropable)) return -1;//buf_size;
> > > +    if(s->last_picture_ptr==NULL && (s->pict_type==B_TYPE || s->dropable)){
> > > +        if(buf2)av_free(buf2);
> > 
> > the if() isnt needed, av_free(NULL) is safe, why ohh why is every single
> > ffmpeg developer adding these checks ;)
> 
> Even better would be to use av_freep everywhere.. right?

maybe, iam not sure ...

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

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is




More information about the ffmpeg-cvslog mailing list