[FFmpeg-devel] [PATCH] lavfi/settb: do not modify the input reference, pass along a new modified copy
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...
Size: 2121 bytes
Desc: not available
More information about the ffmpeg-devel