
<html><head></head><body><div class="gmail_quote">Le 29 juin 2017 15:47:22 GMT+03:00, Francois Cartegnie <fcvlcdev@free.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">Le 29/06/2017 à 14:22, Rémi Denis-Courmont a écrit :<br /><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;"> <br /> It is entirely possible that the FIFO is empty while audio is playing. In fact, this is very likely as audio outputs tend to have large playout buffers, such that the decoder FIFO is most often empty.<br /> <br /> So this looks totally wrong.<br /></blockquote><br />I did not remove aout_IsEmpty().<br /><br />And even if we still had it, DecoderIsEmpty() should test decoder...<br />That's not the case for vout (and that's probably another issue) and<br />wasn't the case for aout.<br />Today's aout only tests draining (EOF) which is wrong and broke menus.<br /><br />Unless we have proper aout_IsEmpty(), assuming the audio is in sync with<br />video, is to consider audio as always empty, or at least, test if<br />decoder has finished its job (what this patch does).<br /><br />Francois<br /><hr /><br />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 clear="all">Audio is never empty until flush or drain has completed. That's how audio works.<br>

-- <br>

Rémi Denis-Courmont<br>

Typed on an inconvenient virtual keyboard</body></html>