<div dir="ltr">By window manager, do you mean at vout_ChangeFullscreen (src\video_output\video_output.c:390)? That's where it starts passing a handle to the window rather than the vout.<div><br></div><div>What was that id parameter originally intended for? I noticed it was changed from a bool to a char* by you last year, but never used.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Oct 13, 2019 at 3:18 AM Rémi Denis-Courmont <<a href="mailto:remi@remlab.net">remi@remlab.net</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">Le sunnuntaina 13. lokakuuta 2019, 6.58.29 EEST Gabriel Luci a écrit :<br>
> Pass the setting as the id parameter to vout_ChangeFullscreen()<br>
> ---<br>
>  src/video_output/vout_intf.c | 6 +++++-<br>
>  1 file changed, 5 insertions(+), 1 deletion(-)<br>
> <br>
> diff --git a/src/video_output/vout_intf.c b/src/video_output/vout_intf.c<br>
> index ce50616fc9..9a0120fb4d 100644<br>
> --- a/src/video_output/vout_intf.c<br>
> +++ b/src/video_output/vout_intf.c<br>
> @@ -566,7 +566,11 @@ static int FullscreenCallback( vlc_object_t *p_this,<br>
> char const *psz_cmd, (void)psz_cmd; (void) oldval; (void)p_data;<br>
> <br>
>      if( newval.b_bool )<br>
> -        vout_ChangeFullscreen(p_vout, NULL);<br>
> +    {<br>
> +        char *psz_fullscreen_monitor = var_InheritString( p_vout,<br>
> "fullscreen-monitor" ); +        vout_ChangeFullscreen(p_vout,<br>
> psz_fullscreen_monitor);<br>
> +        free( psz_fullscreen_monitor );<br>
> +    }<br>
>      else<br>
>          vout_ChangeWindowed(p_vout);<br>
>      return VLC_SUCCESS;<br>
<br>
This breaks the API definition and intended semantics of passing NULL there - <br>
let the window manager pick the most suitable default.<br>
<br>
-- <br>
雷米‧德尼-库尔蒙<br>
<a href="http://www.remlab.net/" rel="noreferrer" target="_blank">http://www.remlab.net/</a><br>
<br>
<br>
<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div>