[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