[FFmpeg-devel] [PATCH v4 7/7] lavc/bsf: make BSF iteration the same as other iterators

[wm4]

On Sun, 4 Feb 2018 22:29:10 +0100
Nicolas George <george at nsup.org> wrote:

> Josh de Kock (2018-02-04):
> > If we were to add in APIs which allowed you to register external components
> > again, this idea wouldn't work well as indexes wouldn't necessarily correspond
> > to the component which it previously did after you register extra components.  
> 
> That is not a problem if the documentation states "index change when
> components are registered".
> 
> > > Another option would be to return the whole list in a mallocated array,
> > > in a single call.  
> > When has exposing more internals ever ended up going well?  
> 
> This suggestion does not expose any internal at all. Are you sure you
> read it correctly?

It would require that all future implementations use an array, until we
deprecate and introduce new APIs again.

I suggest we stay focused, instead of getting nothing done again due to
bikeshedding. Let's just go with the current naming/signature?