
<!DOCTYPE html><html><head><title></title><style type="text/css">p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body><div>That is why code review is important.<br></div><div><br></div><div>We would have said to either fix the "dubious" JSON parser, or remove it (and fix the old JSON parser user to use the new one). We should avoid code duplication at maximum, remember the adaptive HTTP discussions...<br></div><div><br></div><div>And indeed, getting code merged can be a very long and fastidious process. For small patch set, it can take 1 week, sometimes it takes months on bigger ones.<br></div><div><br></div><div>On Mon, Sep 28, 2020, at 14:26, Rémi Denis-Courmont wrote:<br></div><blockquote type="cite" id="qt" style=""><div><a href="https://mailman.videolan.org/pipermail/vlc-devel/2020-September/137937.html">https://mailman.videolan.org/pipermail/vlc-devel/2020-September/137937.html</a><br></div><div><br></div><div class="qt-gmail_quote"><div>Le 28 septembre 2020 15:17:49 GMT+03:00, Francois Cartegnie <fcvlcdev@free.fr> a écrit :<br></div><blockquote class="qt-gmail_quote" 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;"><pre class="qt-k9mail"><div>Le 28/09/2020 à 13:28, Rémi Denis-Courmont a écrit :<br></div><blockquote class="qt-gmail_quote" 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;"><div>Hi,<br></div><div><br></div><div>That was already mentioned in one of the review thread. Also that that code is horrible, with its at best dubious pointer casts, most likely aliasing bugs.<br></div></blockquote><div><br></div><div>????<br></div><div>I see no thread about json except the one from today about the commits.<br></div><div><br></div><div>The last subject mentioning json was on 27/08 about lua/http.<br></div><div><br></div><blockquote class="qt-gmail_quote" 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;"><div>And looking at it again, it seems to decode strings incorrectly to CESU-8 instead of UTF-8.<br></div></blockquote><div><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><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>