
<!DOCTYPE html><html><head><title></title><style type="text/css">p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body><div>What is important is that vout_Create() must be locked by a mutex because you don't want to load 2 "vout window" modules at the same time. It's generally these modules than handle singleton/interface pointer, etc...<br></div><div><br></div><div>Then, vout_Request() could be called unlocked. This will load a "vout display" modules. There can be 2 load at the same time. These modules generally only need the "vout window" context that is always created.<br></div><div><br></div><div>In the past, vout_Request() did a load of "vout window" and "vout display" plugins. That is why it needed to be locked by a mutex.<br></div><div><br></div><div>It is thanks to your job separating vout_Create() and vout_Request() that we can simplify mutexes like I did.<br></div><div><br></div><div>On Wed, May 8, 2019, at 11:47, Rémi Denis-Courmont wrote:<br></div><blockquote type="cite" id="qt"><div>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></div><div><br></div><div class="qt-gmail_quote"><div>Le 8 mai 2019 10:04:52 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 Tue, May 7, 2019, at 20:50, 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 tiistaina 7. toukokuuta 2019, 19.11.43 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>I updated the vout-life/6 branch :<br></div><div><a href="https://code.videolan.org/tguillem/vlc/commits/vout-life/6">https://code.videolan.org/tguillem/vlc/commits/vout-life/6</a><br></div><div><br></div><div>- Big changes in "resource: create first vout from HoldVout*()" and "player:<br></div><div>create vout when requested"<br></div><div><br></div><div> Indeed, you want to create a vout by default, but only when calling<br></div><div>vlc_player_HoldVout*(). So all internal code that hold vouts won't create<br></div><div>one by default.<br></div><div><br></div><div><br></div><div> - "How is the race fixed ?"<br></div><div><br></div><div>I don't really know.<br></div></blockquote><div>If you don't know which of the locking orders you got rid of, then it's pretty <br></div><div>safe to assume that it's none of them and the two locks cannot be merged.<br></div></blockquote><div><br></div><div>I'll try to find it by debuging vlc 3.0<br></div><div>But I'm not sure it's OK to refuse a patch because there might be a bug somewhere. <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>レミ・デニ-クールモン<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>