<!doctype html><html><head><title></title><style type="text/css">p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body>Hi,<br><br>No. Code review would have changed exactly nothing here.<br><br>I was never going to touch or use the awful existing JSON code. It's not just buggy, it's barely legible. And changing existing users is totally out of scope of the YoutubeDL patchset, in every way. What Alexandre said, again.<br><br>And it's pretty rich of somebody who claims to be a nice guy to single me out not once but twice. Congratulations because I'm even more disgusted and even less likely to go through code review onward, than I already was from other people's last week's obstructionism.<br><br><div class="gmail_quote">Le 28 septembre 2020 16:04:00 GMT+03:00, Thomas Guillem <thomas@gllm.fr> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<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></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>