<div dir="ltr">I think you misunderstood my question. This entire patchset was about setting it in the preferences before playback starts. There should be one *setting*, yes. But, from what I see, there are two ways to implement it. (That's what I meant by "2 options")<div><br></div><div>In this patch, I defined a global setting in src/libvlc-module.c. Rémi objected to that approach and said the setting should be defined in the window manager (I assume that means in win32/window.c). But if it's defined there, the setting should be *read* in win32/window.c too.</div><div><br></div><div>My objection to that is that Rémi himself had added an "id" parameter to the vout_ChangeFullscreen() last year, which currently is always NULL. That seems to be the place to pass the ID of the monitor to put the fullscreen window on. If I'm correct on that, then the setting needs to be read in src/video_output/vout_intf.c and thus it should be a global setting.</div><div><br></div><div>And that's where my confusion is. If it really should be a window-provider-specific setting, then why does vout_ChangeFullscreen() have an "id" parameter?</div><div><br></div><div>If it should be in the window provider, that's fine, and I'm happy to do it. There just doesn't seem to be consensus here, so I don't know which direction to go.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Oct 24, 2019 at 11:13 AM Steve Lhomme <<a href="mailto:robux4@ycbcr.xyz">robux4@ycbcr.xyz</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">On 2019-10-24 16:57, Gabriel Luci wrote:<br>
> Following up on this: am I correct in my understanding that those are <br>
> our only 2 options?<br>
<br>
IMO there should be only one option. Either you set it in preferences <br>
before you start playback or you can change dynamically it from the UI. <br>
The second part it not handled for the moment AFAIK.<br>
<br>
> On Thu, Oct 17, 2019 at 5:26 PM Gabriel Luci <<a href="mailto:github@luci.ca" target="_blank">github@luci.ca</a> <br>
> <mailto:<a href="mailto:github@luci.ca" target="_blank">github@luci.ca</a>>> wrote:<br>
> <br>
> I do understand the difference between the two, but they are<br>
> dependent on each other, no?<br>
> <br>
> From my inexperienced perspective, I see two options:<br>
> <br>
> 1. Use a global setting, read it in FullscreenCallback(), and pass<br>
> it to vout_ChangeFullscreen() (which is what I did in this patchset)<br>
> or,<br>
> 2. Define the setting in the window provider plugin, but then that<br>
> means (correct me if I'm wrong) the window provider plugin also has<br>
> the job of reading that setting in its SetFullscreen function. But<br>
> then (I think) that means that there is no purpose for the id<br>
> parameter in vout_ChangeFullscreen(), vout_window_SetFullScreen(),<br>
> and the window provider SetFullscreen functions.<br>
> <br>
> That's why I went with option 1, because I saw the id parameter<br>
> there doing nothing, so I figured that's what it was supposed to be<br>
> for, which led me to believe a global setting was the only way to go.<br>
> <br>
> On Thu, Oct 17, 2019 at 3:44 PM Rémi Denis-Courmont <<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a><br>
> <mailto:<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>>> wrote:<br>
> <br>
> Le torstaina 17. lokakuuta 2019, 19.26.56 EEST Gabriel Luci a<br>
> écrit :<br>
> > I'm happy to make that change. However, then I'm not sure how<br>
> to use the<br>
> > setting. vout_ChangeFullscreen accepts an id parameter, which<br>
> is supposed<br>
> > to be the "output identifier", which I assume means the ID of<br>
> the monitor?<br>
> > (correct me if I'm wrong) So how do we find that value when<br>
> 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<br>
> fullscreen output used<br>
> by an active window which uses vout_ChangeFullscreen(), versus<br>
> changing the<br>
> default fullscreen output for future windows which uses window<br>
> 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><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><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>