<!DOCTYPE html><html><head><title></title><style type="text/css">p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body><div><br></div><div>On Fri, Jul 12, 2019, at 12:03, Rémi Denis-Courmont wrote:<br></div><blockquote type="cite" id="qt"><div>Hi,<br></div><div><br></div><div>None of the objections are specific to multi-track - uselessness, bad performance and extra complexity. And it's very inappropriate to approve a patch if you don't understand why it works or does not (by your own admission) especially if somebody else pointed out problems and doubly so if you sit on the TC.<br></div><div><br></div><div>This is just like block flags all over again. You blindly approve and support whatever François suggests regardless of technical merit and dis-merit.<br></div></blockquote><div><br></div><div>I don't approve this patch in the current state since it's a RFC. Francois just asked us if he was going to the right direction before switching to the low level HTTP API. And I said yes to that (maybe the "LGTM" was confusing).<br></div><div><br></div><div>The final approval/review will be done later when Francois propose the final patch alongside the adaptive switch.<br></div><div><br></div><blockquote type="cite" id="qt"><div><br></div><div class="qt-gmail_quote"><div>Le 12 juillet 2019 08:49:22 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><br></div><div><br></div><div>On Thu, Jul 11, 2019, at 20: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>Le torstaina 11. heinäkuuta 2019, 18.32.20 EEST Thomas Guillem a écrit :<br></div><blockquote style="margin-top:0pt;margin-right:0pt;margin-bottom:1ex;margin-left:0.8ex;border-left-color:rgb(173, 127, 168);border-left-style:solid;border-left-width:1px;padding-left:1ex;" class="qt-gmail_quote"><div>On Mon, Jun 17, 2019, at 10:13, 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(138, 226, 52);border-left-style:solid;border-left-width:1px;padding-left:1ex;" class="qt-gmail_quote"><div>Hi,<br></div><div><br></div><div>There is no explanation why this is needed, and how this won't hurt<br></div><div>performance badly where it's used, or make things more complicated for no<br></div><div>benefits.<br></div><div><br></div><div>I don't disagree that this won't affect normal HTTP streaming, and that's<br></div><div>not the point.<br></div></blockquote><div>Maybe this RFC lack some comments, but we all have guessed that this patch<br></div><div>will be used for adaptive.<br></div></blockquote><div>So what? That does not address any of the three objections above.<br></div></blockquote><div><br></div><div>I see only 2 objections (1 and 3), that I don't really understand, maybe Francois will understand it.<br></div><div><br></div><div>The third one, about multi-track (like AVI) is invalid since this piece of code will only be used by adaptive and there is no multi-track streams in that case.<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<br></div><div><a href="http://www.remlab.net/">http://www.remlab.net/</a><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>