[vlc-commits] vout: always send fullscreen control

[Rémi Denis-Courmont]

vlc | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Sat May 19 21:36:24 2018 +0300| [fbc9feefbbd386124a208d36fa94bc343ac3c4b2] | committer: Rémi Denis-Courmont

vout: always send fullscreen control

Even if the new value for the fullscreen variable matches the old one,
we might still need to update the status.

One case is as follows:
1) Set windowed mode.
2) Set wallpaper mode: this will go to fullscreen without changing the
   fullscreen variable.
3) Leave wallpaper mode.
The window must be returned from fullscreen to windowed mode even though
the fullscreen variable is false all along.

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

 src/video_output/vout_intf.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/video_output/vout_intf.c b/src/video_output/vout_intf.c
index cd6e96f56a..d6a82c521f 100644
--- a/src/video_output/vout_intf.c
+++ b/src/video_output/vout_intf.c
@@ -548,10 +548,9 @@ static int FullscreenCallback( vlc_object_t *p_this, char const *psz_cmd,
                        vlc_value_t oldval, vlc_value_t newval, void *p_data )
 {
     vout_thread_t *p_vout = (vout_thread_t *)p_this;
-    (void)psz_cmd; (void)p_data;
+    (void)psz_cmd; (void) oldval; (void)p_data;
 
-    if( oldval.b_bool != newval.b_bool )
-        vout_ControlChangeFullscreen( p_vout, newval.b_bool );
+    vout_ControlChangeFullscreen( p_vout, newval.b_bool );
     return VLC_SUCCESS;
 }