<div dir="ltr">I see. Do you think it would it make sense to add sanity sleep so that if playlist loops under 1 second the thread would suspend for a while. This would at least prevent the high cpu busy loop on Windows.<div><br></div><div>For demux and decoder there are common cases where it's possible to say file is unplayable (wrong file suffix, garbage content, decoder not supported etc..) but the VLC architecture does not use this kind of error propagation.</div><div><br></div><div>/Teemu<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 20, 2018 at 12:43 PM, Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>AFAICT, the Qt GUI is becoming unresponsive when it displays the error dialog. I do not see any problems with the playlist thread. In fact, there are no issues if the loop is triggered with another interface than Qt.<br>
<br>
And I don't see how the demux or decoder could possibly know that a file cannot be played. That is a very vague notion.<div><div class="h5"><br><br><div class="gmail_quote">Le 20 mars 2018 11:54:41 GMT+02:00, Teemu Ikonen <<a href="mailto:kunteemu@gmail.com" target="_blank">kunteemu@gmail.com</a>> 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 dir="ltr">Hi,<div><br></div><div>When playlist looping is enabled and user opens a video file that is corrupted or codec not supported VLC becomes unresponsive and has to be terminated via Task Manager.</div><div><br></div><div>It appears that the problem is in playlist thread.c that just calls Next immediately after the video has stopped playing (input_dead flag is true) disregarding if file was actually successfully played or not and thus it tries again to play invalid file causing the busy loop.</div><div><br></div><div>Probably there is currently no way for mux/codec to communicate back that file can not be played?</div><div> </div><div>I was thinking fix could be event like INPUT_EVENT_CORRUPTED in addition to existing INPUT_EVENT_DEAD that would allow playlist loop to detect failed file and remove it from the playlist.</div><div><br></div><div>Thanks, Teemu</div><div><br></div><div><br></div><div><br></div><div><br></div></div>
</blockquote></div><br></div></div><span class="HOEnZb"><font color="#888888">
-- <br>
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</font></span></div><br>______________________________<wbr>_________________<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/<wbr>listinfo/vlc-devel</a><br></blockquote></div><br></div></div></div>