[Ffmpeg-cvslog] r6162 - trunk/libavcodec/vc1.c
Rich Felker
dalias
Mon Sep 4 18:04:57 CEST 2006
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?
Rich
More information about the ffmpeg-cvslog
mailing list