
<html xmlns="http://www.w3.org/1999/xhtml">

<head>

<title></title>

</head>

<body>

<div name="messageBodySection">It does seem to be locked somewhere. I have to kill the app but the UI is still responsive.<br />

I never had any similar issue like that before. And just before this patch everything is working fine.</div>

<div name="messageReplySection"><br />

On 26 May 2018 at 08:52 +0200, Rémi Denis-Courmont <remi@remlab.net>, wrote:<br />

<blockquote type="cite">This change just deduplicates the size event handling. The result is the same as before: send a display size change control. The only potential difference that comes to mind is the timing, which might be faster.<br />

<br />

Regardless, sending an event back to your self (through the core) is schizophrenic. There are no reason to call this in a plug-in with windowing.<br />

<br />

Of course, besides potential races within the display plug-in, this also races against the size events of the window provider. But that potential bug is not new, it is also in 3.0.1<br />

<br />

<div class="gmail_quote">Le 26 mai 2018 06:45:38 GMT+02:00, robux4@ycbcr.xyz a écrit :

<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div name="messageBodySection"><font color="#333333">This commit broke Windows playback.<br />

<br />

It doesn't seem to be enough to revert just this one. The define it relied on is gone too.</font></div>

<div name="messageReplySection"><br />

On 20 May 2018 at 19:52 +0200, Rémi Denis-Courmont <git@videolan.org>, wrote:<br />

<blockquote type="cite">vlc | branch: master | Rémi Denis-Courmont <remi@remlab.net> | Fri May 18 21:18:47 2018 +0300| [a4942d980a4d92ab8819b019d47804d43410efb2] | committer: Rémi Denis-Courmont<br />

<br />

display: reroute and deprecate display size event<br />

<br />

The window size should be determined by the window provider, not the<br />

display plugin.<br />

<br />

<blockquote type="cite">http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=a4942d980a4d92ab8819b019d47804d43410efb2<br /></blockquote>

---<br />

<br />

include/vlc_vout_display.h | 7 +++++--<br />

1 file changed, 5 insertions(+), 2 deletions(-)<br />

<br />

diff --git a/include/vlc_vout_display.h b/include/vlc_vout_display.h<br />

index edc1238aee..e70a20ad8e 100644<br />

--- a/include/vlc_vout_display.h<br />

+++ b/include/vlc_vout_display.h<br />

@@ -313,10 +313,13 @@ static inline void vout_display_SendEvent(vout_display_t *vd, int query, ...)<br />

va_end(args);<br />

}<br />

<br />

-static inline void vout_display_SendEventDisplaySize(vout_display_t *vd, int width, int height)<br />

+VLC_DEPRECATED /* Use vout_window_ReportSize() in window provider instead. */<br />

+static inline void vout_display_SendEventDisplaySize(vout_display_t *vd,<br />

+ int width, int height)<br />

{<br />

- vout_display_SendEvent(vd, VOUT_DISPLAY_EVENT_DISPLAY_SIZE, width, height);<br />

+ vout_window_ReportSize(vd->cfg->window, width, height);<br />

}<br />

+<br />

static inline void vout_display_SendEventPicturesInvalid(vout_display_t *vd)<br />

{<br />

vout_display_SendEvent(vd, VOUT_DISPLAY_EVENT_PICTURES_INVALID);<br />

<br />

_______________________________________________<br />

vlc-commits mailing list<br />

vlc-commits@videolan.org<br />

https://mailman.videolan.org/listinfo/vlc-commits<br /></blockquote>

</div>

</blockquote>

</div>

<br />

--<br />

Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté._______________________________________________<br />

vlc-devel mailing list<br />

To unsubscribe or modify your subscription options:<br />

https://mailman.videolan.org/listinfo/vlc-devel<br /></blockquote>

</div>

</body>

</html>