<html><head></head><body>Hi,<br><br>That's not what I meant. I do think we will eventually need a transient state where the window is enabled but there is no display. That state would be used to preserve the window and avoid GUI glitches when changing format, ES or input.<br><br>My point is that I don't think we need, on top of that transient state, a way to bypass it by changing the format. We can just stop the current VD and *then* start a new VD with the new format.<br><br><div class="gmail_quote">Le 14 octobre 2019 11:18:38 GMT+03:00, Thomas Guillem <thomas@gllm.fr> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<pre class="k9mail"><br><br>On Sat, Oct 12, 2019, at 13:41, Rémi Denis-Courmont wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">Le perjantaina 11. lokakuuta 2019, 16.33.18 EEST Steve Lhomme a écrit :<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;">vout_Close does vout_StopDisplay + vout_DisableWindow<br><br>In the case where there is no display (vout_StopDisplay not called) we<br>should still disable the window before destroying it.<br></blockquote>At least historically, vout_Stop() would yield an assertion failure if the <br>display was missing. And that's very much on purpose - to forbid sloppy <br>programming calling vout_Stop() on an already stopped/not-started vout.<br><br>This patch should not work - or something regressed.<br></blockquote><br>Like rémi, I don't think we should have a state where the VD is stopped but the windows is still enabled (except when restarting, cf. vout_Request).<br><br>> <br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">-- <br>レミ・デニ-クールモン<br><a href="http://www.remlab.net/">http://www.remlab.net/</a><hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></blockquote><hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a></pre></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>