[vlc-devel] [PATCH 2/2] es: replace i_original_channels with i_stereo_mode
thomas at gllm.fr
Tue Jul 11 15:58:08 CEST 2017
On Tue, Jul 11, 2017, at 15:29, Rémi Denis-Courmont wrote:
> On mardi 11 juillet 2017 10:01:34 EEST Thomas Guillem wrote:
> > The i_original_channels value (that is now i_stereo_mode) should not be used
> > by aout modules, and shouldn't be set by input if there is no special
> > stereo mode.
> > If accepeted, this commits will be splitted in the following way:
> > - One commit per module that remove the usage of i_original_channels when
> > not needed at all.
> > - One big commit that replace i_original_channels with i_stereo_mode and
> > that changes few modules (mpg/a52/dca/dts/filters/core).
> I have tried to move the original channels to a proper set of flags
> But I failed everytime, because I never understood what the channel mask
> within original channels was exactly meant to be. As far as I could tell,
> was simply three flags with the channels identical to the physical mask
> Also, if you want to rationalize this, there is really only one flag that
> belongs there (Dolby). Reverse stereo could just as well be fixed by
> in the decoder (easy). And in principles, dual-mono should be two
> audio ES's (less easy).
You mean, reodering in the audio_output code, just between the filters
and the output ?
I have a branch that allow users to force a stereo-mode even with non
stereo input. Some people asked this feature, I guess some content are
better downmixed to stereo than rendered on a 5.1 for example.
I can have a look for reverse-stereo, this mode is only asked by the
user will never come from the input.
I would also like to add the Binauralizer here.
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
More information about the vlc-devel