[FFmpeg-devel] [PATCH 02/12] lavf: update auto-bsf to new BSF API

[Nicolas George]

Le decadi 10 floréal, an CCXXIV, Rodger Combs a écrit :
> There's definitely going to be some trickiness around this if we end up
> having 1:N filters, or filters that actually requiring flushing.
> Otherwise, though, this should work fine, and we can address multiple-bsf
> or more-advanced BSF cases when they come up.

> pkt can't be NULL here. Flushing the filters will be somewhat complex
> (requiring a loop over all the streams to flush them all), and AFAIK none
> of the current automatically-added BSFs require it, so I'm punting on that
> for now.

> This also doesn't happen with any current BSFs, and the required looping
> is non-trivial, so I'm punting on this as well. Adding TODO comments for
> both issues.

With comments stating the current limitations for future reference, no
objection from me.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160505/54adef99/attachment.sig>