<html><head></head><body>I'm not sure it's okay to propose patches with the same known bug four times in a row, make the reviewer repeat the problem twice as many, and then imply that the reviewer may be misbehaving.<br><br><div class="gmail_quote">Le 8 mai 2019 10:04:52 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;">
<pre class="k9mail"><br><br>On Tue, May 7, 2019, at 20:50, Rémi Denis-Courmont wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">Le tiistaina 7. toukokuuta 2019, 19.11.43 EEST Thomas Guillem a écrit :<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;">I updated the vout-life/6 branch :<br><a href="https://code.videolan.org/tguillem/vlc/commits/vout-life/6">https://code.videolan.org/tguillem/vlc/commits/vout-life/6</a><br><br>- Big changes in "resource: create first vout from HoldVout*()" and "player:<br>create vout when requested"<br><br> Indeed, you want to create a vout by default, but only when calling<br>vlc_player_HoldVout*(). So all internal code that hold vouts won't create<br>one by default.<br><br><br> - "How is the race fixed ?"<br><br>I don't really know.<br></blockquote>If you don't know which of the locking orders you got rid of, then it's pretty <br>safe to assume that it's none of them and the two locks cannot be merged.<br></blockquote><br>I'll try to find it by debuging vlc 3.0<br>But I'm not sure it's OK to refuse a patch because there might be a bug somewhere. <br><br>> <br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">-- <br>レミ・デニ-クールモン<br><a href="http://www.remlab.net/">http://www.remlab.net/</a><hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></blockquote><hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a></pre></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>