<div dir="ltr"><div>I do understand the difference between the two, but they are dependent on each other, no?</div><div><br></div><div>From my inexperienced perspective, I see two options:</div><div><br></div><div>1. Use a global setting, read it in FullscreenCallback(), and pass it to vout_ChangeFullscreen() (which is what I did in this patchset)</div><div>or,</div><div>2. Define the setting in the window provider plugin, but then that means (correct me if I'm wrong) the window provider plugin also has the job of reading that setting in its SetFullscreen function. But then (I think) that means that there is no purpose for the id parameter in vout_ChangeFullscreen(), vout_window_SetFullScreen(), and the window provider SetFullscreen functions.</div><div><br></div><div>That's why I went with option 1, because I saw the id parameter there doing nothing, so I figured that's what it was supposed to be for, which led me to believe a global setting was the only way to go.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Oct 17, 2019 at 3:44 PM 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 torstaina 17. lokakuuta 2019, 19.26.56 EEST Gabriel Luci a écrit :<br>
> I'm happy to make that change. However, then I'm not sure how to use the<br>
> setting. vout_ChangeFullscreen accepts an id parameter, which is supposed<br>
> to be the "output identifier", which I assume means the ID of the monitor?<br>
> (correct me if I'm wrong) So how do we find that value when calling<br>
> vout_ChangeFullscreen if only the window provider knows? Or am I<br>
> misunderstanding something there?<br>
<br>
I think you're mixing up two things here - changing the fullscreen output used <br>
by an active window which uses vout_ChangeFullscreen(), versus changing the <br>
default fullscreen output for future windows which uses window provider-<br>
dependent mechanism - a setting in Qt and XDG-shell cases.<br>
<br>
-- <br>
Rémi Denis-Courmont<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>