[vlc-devel] [PATCH 4/4] win32: Send fullscreen to specific monitor

Alexandre Janniaux ajanni at videolabs.io
Sat Oct 26 10:44:34 CEST 2019


Hi,

I'm sorry if it was confusing for you. I meant to wrote
that I agree with you, but this fact and enumeration for
CLI/GUI wasn't stated in the documentation.

But it is easily fixable.

Regards,
--
Alexandre Janniaux
Videolabs

On Sat, Oct 26, 2019 at 05:18:15AM +0300, Rémi Denis-Courmont wrote:
> Hi,
>
> Hot plug is not optional. It's not needed for Qt indeed, but it is needed for LibVLC apps. We already had that argument with audio devices.
>
> Le 25 octobre 2019 22:14:00 GMT+03:00, Alexandre Janniaux <ajanni at videolabs.io> a écrit :
> >Probably not bugs, with regards to the documentation for
> >this which is incomplete, but I agree with Rémi that it
> >should also work with caching when the module is
> >initialized, because it should report new or removed
> >outputs.
> >
> >When we're doing the enumeration from the UI, it doesn't
> >matter and we shouldn't do caching as there is not enough
> >lifetime for interesting storage anyway. As it is only for
> >configuration it almost doesn't matter in this design.
> >
> >However in your scenario, the window will indeed not do the
> >correct behaviour with the current window implementation,
> >as the fullscreen state and information is not stored and
> >restored during Enable. That's a very good point.
> >
> >The pivotal point here is that requests like SetFullscreen
> >can be applied with the window being disabled. It's not
> >very documented but the main motive behind is to be able to
> >configure the video playback surface before actually making
> >it visible to the user, while keeping the window provider
> >module running so as to reports outputs. It's a bit
> >confusing from a module provider implementor point of view
> >but is far easier to use from the API user point of view
> >and provides a better UX than in previous 3.0.
> >
> >We probably want something like not changing screen if the
> >video is already started, or if it goes to the next one,
> >but the interface might want to set the correct screen when
> >the playback will be stopped.
> >
> >Regards,
> >--
> >Alexandre Janniaux
> >Videolabs
> >
> >On Fri, Oct 25, 2019 at 08:49:03PM +0300, Rémi Denis-Courmont wrote:
> >> Le perjantaina 25. lokakuuta 2019, 20.37.06 EEST Gabriel Luci a écrit
> >:
> >> > I would normally agree, but monitors can be unplugged and plugged
> >in at any
> >> > time (especially wireless displays). Consider this scenario:
> >> >
> >> > 1. A chosen monitor is selected and VLC closed.
> >> > 2. The chosen monitor is disconnected.
> >> > 3. The next time VLC is run, and video played, the enumeration
> >(without the
> >> > chosen monitor) is saved, and video plays on the primary monitor.
> >> > 4. You realize your forgot to plug in your chosen monitor, and do
> >so.
> >> >
> >> > In that case, VLC will not play video on the chosen monitor until
> >the
> >> > enumeration is forced to happen again (restarting VLC or opening
> >> > preferences, for example).
> >>
> >> Sure it will. Unless you write bugs.
> >>
> >> --
> >> Rémi Denis-Courmont
> >> http://www.remlab.net/
> >>
> >>
> >>
> >> _______________________________________________
> >> vlc-devel mailing list
> >> To unsubscribe or modify your subscription options:
> >> https://mailman.videolan.org/listinfo/vlc-devel
> >_______________________________________________
> >vlc-devel mailing list
> >To unsubscribe or modify your subscription options:
> >https://mailman.videolan.org/listinfo/vlc-devel
>
> --
> Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.

> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list