On 2015-04-28 10:17, Tanu Kaskinen wrote: > 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. Thanks. > 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. No strong preference here - if you're planning to merge pactl's volume handling with pa_parse_volume too, sounds like you would end up doing it anyway? -- David Henningsson, Canonical Ltd. https://launchpad.net/~diwic
