<html><head></head><body>Hi,<br><br>The point is that pause/resume was added because some accesses needed it, and they don't expect data to flow during pause.<br><br>Furthermore, we can't allow reading during pause as it breaks live streaming. Otherwise, you'd get a huge hysteresis on pause/resume as you'd fill the buffer on pause and drain it on resume.<br><br><div class="gmail_quote">Le 18 novembre 2019 23:27:26 GMT+02:00, Alexandre Janniaux <ajanni@videolabs.io> 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">Hi,<br><br>On Mon, Nov 18, 2019 at 10:07:22PM +0200, 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 maanantaina 18. marraskuuta 2019, 19.13.03 EET Alexandre Janniaux a écrit :<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;"> Hi,<br><br> On Mon, Nov 18, 2019 at 05:56:43PM +0200, Rémi Denis-Courmont wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #8ae234; padding-left: 1ex;">I don't see how the error message is wrong. It is not permissible to read<br>from a paused stream, by definition. This patch looks wrong to me.<br></blockquote>The concept of a playback pipeline being paused is fairly simple<br>considering the user point of view, but from the internal point of<br>view I would have expected that pause is just relative to audio/video<br>playback itself, not everything before which would be paced by the<br>pool of available resources.<br></blockquote>The concept of pause is easy. Data does not flow during pause. There are no<br>other reasons why pause was exposed to stream filters and accesses in the first<br>place.<br></blockquote><br>Sure, I hear that, it is not really a hint about why though.<br><br>I should have phrase my sentence like the following:<br><br>If we want data to not flow during pause, it can be achieved<br>by two effects:<br>+ we don't process data anymore, the whole pipeline is<br>freezed.<br>+ we don't switch the last frame / don't display audio, the<br>pipeline process is stalled because paced.<br><br>What Thomas is doing is the second one in this patch. I<br>tested it and it allows seeking while pausing and brings a<br>better UX.<br><br>The two questions then:<br>+ if we keep doing the first, how pause should be handled to<br>allow frame by frame ?<br>+ if we're going for the second, what are the setbacks?<br><br>Do you have any hints with regards to these ?<br><br>Regards,<br>--<br>Alexandre Janniaux<br>Videolabs<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>