[FFmpeg-devel] [PATCH] vsrc_buffer: remove dependency on AVFrame

Stefano Sabatini stefano.sabatini-lala at poste.it
Thu May 19 23:29:42 CEST 2011


On date Thursday 2011-05-19 16:47:29 +0200, Michael Niedermayer encoded:
> On Thu, May 19, 2011 at 12:29:18PM +0200, Stefano Sabatini wrote:
> > On date Saturday 2011-05-14 15:05:12 +0200, Stefano Sabatini encoded:
> > > On date Saturday 2011-05-14 13:33:40 +0200, Michael Niedermayer encoded:
> > > > On Sat, May 14, 2011 at 12:04:45PM +0200, Stefano Sabatini wrote:
> > [...]
> > > > > > > What about:
> > > > > > > 
> > > > > > > av_vsrc_buffer_add_frame2(ost->input_video_filter, avfilter_get_video_buffer_ref_from_frame(frame, perm), "");
> > > > > > > ?
> > > > > > 
> > > > > > this looks much better
> > > 
> > > I didn't consider that the picref needs to be unreffed, new patches
> > > without memleak (and two lines of more code for freeing the allocated
> > > picref without to free the frame data, which is managed by the
> > > decoder).
> > 
> > New patchset, I'm adding another wrapper function for directly pushing
> > AVFrame data.
> 
> I like this patchset alot
> if it works and has been tested i think it should be pushed

Pushed.
-- 
FFmpeg = Fascinating & Fast Multimedia Pacific ExchanGer


More information about the ffmpeg-devel mailing list