]> git.sesse.net Git - vlc/commitdiff
aout: remove what should be an unneeded update of "mute"
authorErwan Tulou <erwan10@videolan.org>
Wed, 20 Mar 2013 00:16:11 +0000 (01:16 +0100)
committerErwan Tulou <erwan10@videolan.org>
Thu, 21 Mar 2013 15:04:35 +0000 (16:04 +0100)
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

src/playlist/aout.c

index 72c7f0ff40f43f43ba1ddb7af8c446769a0d4466..5a45515b7dbb1ff56271f6410285f1162fed2869 100644 (file)
@@ -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;
 }