[pulseaudio-discuss] [PATCH v3 4/6] conf-parser: Add pa_config_parse_volume()

Tanu Kaskinen tanu.kaskinen at linux.intel.com
Tue Apr 28 01:17:32 PDT 2015


On Tue, 2015-04-28 at 09:39 +0200, David Henningsson wrote:
> 
> On 2015-04-27 13:34, Tanu Kaskinen wrote:
> > This will be used to parse the path default volume in the alsa mixer
> > configuration files. For now, percentages are the only supported
> > format, because I don't have need for anything else, but we can of
> > course add more volume formats later as needed.
> 
> My preference for allowing all volume formats is quite strong; 
> specifically considering that volume percentages are frowned upon as 
> meaning different things in different contexts (so newcomers to 
> PulseAudio don't have to figure out what *we* mean with it before being 
> able to write the config).

Ok, I'll support dB and raw pa_volume_t too, then. Should I also add
support for linear multiplication factors for completeness? pactl parses
"0.5" as a multiplier that gets applied to each sample, but
pa_parse_volume() doesn't support that currently.

-- 
Tanu



More information about the pulseaudio-discuss mailing list