<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jan 10, 2020 at 5:43 PM Thomas Guillem <<a href="mailto:thomas@gllm.fr">thomas@gllm.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><u></u><div><div>Alexandre, ideally we would like to avoid variables to control vouts, but it's not done yet.<br></div><div>So, in the meantime we should continue to use variables.<br></div><div><br></div><div>Akshay, you spotted the problem, about the vout/player var_Set mismatch. I think the correct way to fix this issue is to fix the vlc_player_vout_IsFullscreen() function. This function should return the state of the first vout (via vlc_player_vout_Hold()) instead of the player state.<br></div><div>Same for vlc_player_vout_IsWallpaperModeEnabled().<br></div><div><br></div><div>You could also  remove<br></div><div><br></div><div>    /* vout variables */<br></div><div>    VAR_CREATE("fullscreen", VLC_VAR_BOOL | VLC_VAR_DOINHERIT);<br></div><div>    VAR_CREATE("video-on-top", VLC_VAR_BOOL | VLC_VAR_DOINHERIT);<br></div><div>    VAR_CREATE("video-wallpaper", VLC_VAR_BOOL | VLC_VAR_DOINHERIT);<br></div><div><br></div><div>in src/player/player.c<br></div><div><br></div></div></blockquote><div>should the removal of the vout variables and the updating the functions be in a single commit or not and if in two commits should this be in a single patch file?</div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr">Regards,<div>Akshay V</div><div><a href="http://github.com/akshayaky" target="_blank">GitHub</a> | Amrita Vishwa Vidyapeetham<br></div></div></div></div></div></div>