[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