[vlc-commits] aout: remove what should be an unneeded update of "mute"

Erwan Tulou git at videolan.org
Thu Mar 21 16:11:25 CET 2013


vlc | branch: master | Erwan Tulou <erwan10 at videolan.org> | Wed Mar 20 01:16:11 2013 +0100| [2ad2e671b814ebcee2a4724baf5a8606a5938554] | committer: Erwan Tulou

aout: remove what should be an unneeded update of "mute"

if aout_MuteSet returns successfully, the "mute" variables at the aout
and playlist levels have already been updated via a call to aout_MuteReport.

This late update was a hack to make things work when a audio output driver
forgets to call aout_MuteReport. So this patch may bring regression for those
audio output drivers that still forget to call aout_MuteReport and yet
terminate successfully. This is likely to occur in corner cases, for instance
when no stream is available as was the case for pulse.c

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=2ad2e671b814ebcee2a4724baf5a8606a5938554
---

 src/playlist/aout.c |    2 --
 1 file changed, 2 deletions(-)

diff --git a/src/playlist/aout.c b/src/playlist/aout.c
index 72c7f0f..5a45515 100644
--- a/src/playlist/aout.c
+++ b/src/playlist/aout.c
@@ -120,8 +120,6 @@ int playlist_MuteSet (playlist_t *pl, bool mute)
     {
         ret = aout_MuteSet (aout, mute);
         vlc_object_release (aout);
-        if (ret == 0)
-            var_SetBool (pl, "mute", mute);
     }
     return ret;
 }



More information about the vlc-commits mailing list