diff options
| author | Clemens Ladisch <clemens@ladisch.de> | 2010-12-06 14:05:10 +0100 |
|---|---|---|
| committer | Clemens Ladisch <clemens@ladisch.de> | 2010-12-06 14:05:10 +0100 |
| commit | 70a01748d594ef57a1962e4cb012927faf6d852f (patch) | |
| tree | 79be0849a2507ee5802563c7815d4c0f24564036 /alsamixer | |
| parent | e57f619c82e2fec20c20c30f56d3fa4db22a8353 (diff) | |
alsamixer: increase step size for big control value ranges
For controls with a big range, stepping through all values can become
tedious and make it impossible to adjust the volume easily. Therefore,
ensure that all steps are big enough so that the full range has at most
one hundred steps.
Signed-off-by: Clemens Ladisch <clemens@ladisch.de>
Diffstat (limited to 'alsamixer')
| -rw-r--r-- | alsamixer/mixer_widget.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/alsamixer/mixer_widget.c b/alsamixer/mixer_widget.c index c8ca156..adb4568 100644 --- a/alsamixer/mixer_widget.c +++ b/alsamixer/mixer_widget.c @@ -318,7 +318,7 @@ static void change_volume_to_percent(struct control *control, int value, unsigne set_func(control->elem, control->volume_channels[1], min + (max - min) * value / 100); } -static void change_volume_relative(struct control *control, int delta, unsigned int channels) +static void change_volume_relative(struct control *control, long delta, unsigned int channels) { int (*get_range_func)(snd_mixer_elem_t *, long *, long *); int (*get_func)(snd_mixer_elem_t *, snd_mixer_selem_channel_id_t, long *); @@ -352,6 +352,8 @@ static void change_volume_relative(struct control *control, int delta, unsigned if (err < 0) return; } + if (max - min > 100) + delta = (delta * (max - min) + (delta > 0 ? 99 : -99)) / 100; if (channels & LEFT) { value = left + delta; if (value < min) |