[vlc-devel] Regarding bug #3913 (cli volume option)

[Rémi Denis-Courmont]

Le samedi 19 mars 2011 17:34:06 Francois Cartegnie, vous avez écrit :
> Hi,
> 
> 
> The audio command line --volume is broken.
> vlc --intf=dummy --volume=0 foobar.mp3
> 
> - The command line option is correcly parsed
> - The aout output instance has been set to vlcrc value
> - No call aoutSetVolume() is made from cli/dummy
> 
> Seems there's a prefs/command line var priority problem when the aout_ouput
> retrieves its value.
> 
> Here follows a test late fix that enlightens the problem:
> - var_inherit(volume) gets command line or defaults to vlcrc.
> - forcing the value again fixes the volume problem.
> 
> Anyone knows where to look to (correctly) fix the problem earlier ?

This is part of a wider problem with audio output parameters not being scoped 
correctly. They should be stored as variables on the audio output object, of 
which there should be one per input manager.

Currently, those core parameters (notably volume) are stored in the 
configuration. This "breaks" not only the command line, but also multiple 
LibVLC inputs with independent volume controls.
-- 
Rémi Denis-Courmont
http://www.remlab.info/
http://fi.linkedin.com/in/remidenis