fade: Handle non-positive fade time gracefully.
authorAndre Noll <maan@systemlinux.org>
Sun, 15 Jul 2012 21:57:53 +0000 (23:57 +0200)
committerAndre Noll <maan@systemlinux.org>
Sun, 7 Oct 2012 09:35:38 +0000 (11:35 +0200)
Currently, we don't do anything if the fade_time parameter is less
than or equal to zero. Just setting the destination volume seems to
be more natural in this case.

fade.c

diff --git a/fade.c b/fade.c
index 2954fa3..68e1b5f 100644 (file)
--- a/fade.c
+++ b/fade.c
@@ -52,7 +52,7 @@ static int fade(struct mixer *m, struct mixer_handle *h, int new_vol, int fade_t
        unsigned long long tmp, tmp2; /* Careful with that axe, Eugene! */
 
        if (fade_time <= 0)
        unsigned long long tmp, tmp2; /* Careful with that axe, Eugene! */
 
        if (fade_time <= 0)
-               return 1;
+               return m->set(h, new_vol);
        secs = fade_time;
        PARA_NOTICE_LOG("fading to %d in %d seconds\n", new_vol, secs);
        ret = m->get(h);
        secs = fade_time;
        PARA_NOTICE_LOG("fading to %d in %d seconds\n", new_vol, secs);
        ret = m->get(h);