[vlc-devel] [RFC PATCH 17/17] spdif: set "spdif" option true by default

[Thomas Guillem]

On Wed, Jul 27, 2016, at 10:23, Rémi Denis-Courmont wrote:
> Le 2016-07-25 18:58, Jean-Baptiste Kempf a écrit :
> > On 25 Jul, Rémi Denis-Courmont wrote :
> >> What prevents S/PDIF by default on unmanaged audio outputs, such as 
> >> ALSA?
> >
> > Don't they need their own option?
> 
> Probably.
> 
> Some outputs cannnot "negotiate" S/PDIF support. They just take it as 
> if it were a signed 16-bits PCM stream, and assume that the wiring is 
> accordingly. ALSA is among them. Those outputs need a 
> disabled-by-default option.
> 
> Other outputs negotiate S/PDIF at the API level, with either the link 
> layer (e.g. HDMI) or OS configuration as back-end. They need either no 
> option or a mostly redundant enabled-by-default option. WASAPI, 
> PulseAudio come to mind.
> 
> And the remaining ones just don't support S/PDIF at all.
> 
> So I think the notion of a global decoder-level option is doomed, since 
> there would be contradictory requirements for the default value.

So I let the "spdif" option like before. This option will be checked
only for aouts where S/PDIF should be disabled by default (ALSA).

Maybe, I can set the "spdif" option an advanced one ?

> 
> -- 
> Rémi Denis-Courmont
> http://www.remlab.net/
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel