]> git.sesse.net Git - vlc/commit
"fullscreen" callback: do nothing if value is unchanged
authorRémi Denis-Courmont <remi@remlab.net>
Wed, 10 Feb 2010 16:03:47 +0000 (18:03 +0200)
committerRémi Denis-Courmont <remi@remlab.net>
Wed, 10 Feb 2010 16:43:29 +0000 (18:43 +0200)
commit49334e07d05bb5e4770a95b8b4f57d84c192cdc9
tree6fb9d9c69b45ad4bdfaa4d0b002fb153e90aa850
parentec104981f5e337d321520a5ba7adf9e29892be23
"fullscreen" callback: do nothing if value is unchanged

The old video output core assumes that the fullscreen state must be
toggled if the VOUT_FULLSCREEN_CHANGE bit is set. So we need to check
that the target state is not already correct, as the VLC variables core
does NOT do it internally. This commit provides rare exception to the
rule that oldval is useless.

This fixes a whole class of race conditions where two threads try to
change fullscreen status at the same time.

In the video filter case, we now enable fullscreen on all childrens, not
just one. This seems a bit more logical for wall. Without this, toggling
fullscreen would only ever work on the first video output. With this,
things should work great if the different pieces of the wall are on
different video ports, and OK (Alt+Tab is your friend) otherwise.
modules/control/hotkeys.c
modules/video_filter/filter_common.h
modules/video_filter/wrapper.c
src/video_output/event.h
src/video_output/vout_intf.c