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

Michael Niedermayer michaelni at gmx.at
Thu May 19 16:47:29 CEST 2011


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

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many that live deserve death. And some that die deserve life. Can you give
it to them? Then do not be too eager to deal out death in judgement. For
even the very wise cannot see all ends. -- Gandalf
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110519/5f9aaa40/attachment.asc>


More information about the ffmpeg-devel mailing list