<!DOCTYPE html><html><head><title></title><style type="text/css">
p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body><div>Hello,<br></div><div><br></div><div>it's hard to review a .patch file.<br></div><div><br></div><div>Please send your patch inline or send a link to a gitlab/github MR/branch.<br></div><div><br></div><div>I would really prefer if we remove the old one when adding this new one. If you are not sure libRIST is built on all platforms supported by VLC, just do a merge request to your own repo, with contrib librist, and the librist module in order to trigger the CI. That way, we will know which platforms need fixes.<br></div><div><br></div><div>On Wed, Feb 3, 2021, at 02:43, Sergio M. Ammirata, Ph.D. wrote:<br></div><blockquote type="cite" id="qt" style="text-align:left;direction:ltr;"><div>Any feedback?<br></div><div><br></div><div>On Thu, 2021-01-14 at 22:22 -0500, Sergio M. Ammirata, Ph.D. wrote:<br></div><blockquote type="cite" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-color:rgb(114, 159, 207);border-left-style:solid;border-left-width:2px;padding-left:1ex;"><div><br></div><div>Here is a revised patch with compatible config settings (all items below fixed). I also reduced the number of settings to the minimum required.<br></div><div><br></div><div>Regards,<br></div><div><br></div><div>Sergio<br></div><div><br></div><div>On Sun, 2020-12-27 at 16:39 +0100, Jean-Baptiste Kempf wrote:<br></div><blockquote type="cite" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-color:rgb(114, 159, 207);border-left-style:solid;border-left-width:2px;padding-left:1ex;"><div>On Sun, 27 Dec 2020, at 14:15, Sergio M. Ammirata, Ph.D. wrote:<br></div><blockquote type="cite" id="qt-qt" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-color:rgb(114, 159, 207);border-left-style:solid;border-left-width:2px;padding-left:1ex;"><div>The config options do not match 100% either. Here is the analysis on the config options:<br></div><div>1) The config for the buffer size is called "latency" on native rist and "buffer" in the librist module.<br></div></blockquote><div><br></div><div>Fix this.<br></div><div><br></div><blockquote type="cite" id="qt-qt" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-color:rgb(114, 159, 207);border-left-style:solid;border-left-width:2px;padding-left:1ex;"><div>2) There are a few config options present in native rist that do not exist on the librist module (packet-size, maximum-jitter, nack-type, disable-nacks, mcast-blind-nacks).<br></div></blockquote><div><br></div><div>Fix this.<br></div><div><br></div><blockquote type="cite" id="qt-qt" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-color:rgb(114, 159, 207);border-left-style:solid;border-left-width:2px;padding-left:1ex;"><div>3) There are a number of new config options present in librist module that do not exist in native rist. These are expected as the new settings/options are related to encryption, selecting profile levels, multiplexing, multipath, congestion control and timing mode.<br></div></blockquote><div><br></div><div>Remove the unneeded ones. We have too many already.<br></div><div><br></div><div>Best,<br></div><div><br></div><div id="qt-sig60240713"><div class="qt-signature">--<br></div><div class="qt-signature">Jean-Baptiste Kempf - President<br></div><div class="qt-signature">+33 672 704 734<br></div><div class="qt-signature"> <br></div><div class="qt-signature"><br></div></div><div><br></div></blockquote><pre>_______________________________________________<br></pre><pre>vlc-devel mailing list<br></pre><pre>To unsubscribe or modify your subscription options:<br></pre><pre><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></pre></blockquote><div>_______________________________________________<br></div><div>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><br></div></body></html>