[vlc-devel] [PATCH 04/12] input/event: add input_SendDecoderState

[Rémi Denis-Courmont]

On Thursday 09 June 2016 19:26:49 Thomas Guillem wrote:
> On Thu, Jun 9, 2016, at 12:03, Rémi Denis-Courmont wrote:
> > I think that the decoder state is an implementation detail and should
> > not be exposed beyond ES output. And it definitely must not be exposed
> > to LibVLC as implementation details should not be set in stone through
> > the public API.
> 
> But this set of patches was done for the public API.
> Is it OK if there is only one type of error ? So there will be 2 states
> (for now):
> 
> DECODER_STATE_HW
> DECODER_STATE_ERROR

My point is that decoders are an implementation detail. You can maybe mark a 
track/ES as failed. But even then, exposing that in a meaningful and backward-
compatible manner seems difficult, as apps tend to assume that all enumerated 
tracks are available.

-- 
Rémi Denis-Courmont
http://www.remlab.net/