<!DOCTYPE html><html><head><title></title><style type="text/css">p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body><div><br></div><div>On Mon, Jun 17, 2019, at 10:13, Rémi Denis-Courmont wrote:<br></div><blockquote type="cite" id="qt"><div>Hi,<br></div><div><br></div><div>There is no explanation why this is needed, and how this won't hurt performance badly where it's used, or make things more complicated for no benefits.<br></div><div><br></div><div>I don't disagree that this won't affect normal HTTP streaming, and that's not the point.<br></div></blockquote><div><br></div><div>Maybe this RFC lack some comments, but we all have guessed that this patch will be used for adaptive.<br></div><div><br></div><div>cf. <a href="https://trac.videolan.org/vlc/ticket/22339">https://trac.videolan.org/vlc/ticket/22339</a><br></div><div><br></div><div>I think Francois is doing what you suggested: using and modifying the lower HTTP API.<br></div><div><br></div><div>So LGTM, but you should add some rational in documentation or commit logs.<br></div><div><br></div><div><br></div><blockquote type="cite" id="qt"><div><br></div><div class="qt-gmail_quote"><div>Le 17 juin 2019 09:23:19 GMT+03:00, Thomas Guillem <thomas@gllm.fr> a écrit :<br></div><blockquote style="margin-top:0pt;margin-right:0pt;margin-bottom:0pt;margin-left:0.8ex;border-left-color:rgb(204, 204, 204);border-left-style:solid;border-left-width:1px;padding-left:1ex;" class="qt-gmail_quote"><pre class="qt-k9mail"><div>hi,<br></div><div><br></div><div>On Sat, Jun 15, 2019, at 07:53, Rémi Denis-Courmont wrote:<br></div><blockquote style="margin-top:0pt;margin-right:0pt;margin-bottom:1ex;margin-left:0.8ex;border-left-color:rgb(114, 159, 207);border-left-style:solid;border-left-width:1px;padding-left:1ex;" class="qt-gmail_quote"><div> Hi,<br></div><div><br></div><div>Streaming in ranges breaks pipelining and cannot work properly if the RTT is <br></div><div>non-negligible. Sequential streaming must use a half-open range.<br></div><div><br></div><div>Likewise multi-track (like AVI) streaming needs to use multiple half-open <br></div><div>ranges in parallel, in other words, multiple streams - and that is not specific <br></div><div>to HTTP at all.<br></div></blockquote><div><br></div><div>Range request will be only used for adaptive, why are talking about .avi ?<br></div><div><br></div><div>> <br></div><blockquote style="margin-top:0pt;margin-right:0pt;margin-bottom:1ex;margin-left:0.8ex;border-left-color:rgb(114, 159, 207);border-left-style:solid;border-left-width:1px;padding-left:1ex;" class="qt-gmail_quote"><div>New stream must only be used when unavoidable - notably when seeking or <br></div><div>getting a different resource (e.g. rate switch).<br></div><div><br></div><div>This patch has zero rationale. Nack.<br></div></blockquote><div><br></div><div>This patch doesn't change the actual http access behavior. This new function will only be used by adaptive in order to refactor *a lot* of code.<br></div><div><br></div><div>I really don't understand your refusal ? Could you explain what should be done then ?<br></div><div><br></div><div>> <br></div><blockquote style="margin-top:0pt;margin-right:0pt;margin-bottom:1ex;margin-left:0.8ex;border-left-color:rgb(114, 159, 207);border-left-style:solid;border-left-width:1px;padding-left:1ex;" class="qt-gmail_quote"><div>-- <br></div><div>Rémi Denis-Courmont<hr>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></div></blockquote><div><hr>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></div></pre></blockquote></div><div><br></div><div>-- <br></div><div>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté. <br></div><div>_______________________________________________<br></div><div>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div>https://mailman.videolan.org/listinfo/vlc-devel<br></div></blockquote><div><br></div></body></html>