[FFmpeg-devel] [PATCH] lavfi/settb: do not modify the input reference, pass along a new modified copy

Stefano Sabatini stefasab at gmail.com
Mon Jul 30 16:57:28 CEST 2012


On date Monday 2012-07-30 16:29:44 +0200, Nicolas George encoded:
> Le tridi 13 thermidor, an CCXX, Stefano Sabatini a écrit :
> > Avoid to modify the internal properties which may be accessed by another
> > filter before in the filterchain, in case that is not caching an internal
> > reference and needs to access the unaltered data later after start_frame().
> 
> I think it does not work, because of that:
> 
> > +        avfilter_unref_buffer(inpicref);
> 
> A filter that accesses a ref after having given it away is bogus and
> absolutely needs to be fixed.
> 
> Therefore, I think there is absolutely no need to allocate a new reference
> (which is not completely cheap).
> 
> > Also improve consistency with filter_samples() of asettb, allowing a
> > pending factorization.
> 
> I am all for the factorization, but doing it the other way around seems a
> better idea.

Works for me.
-- 
FFmpeg = Foolish and Fanciful Meaningful Power Enlightened Game
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-lavfi-asettb-do-not-create-a-new-reference.patch
Type: text/x-diff
Size: 2121 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120730/4ea40280/attachment.bin>


More information about the ffmpeg-devel mailing list