<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le lun. 20 sept. 2021 à 19:32, Bastien ROUCARIES <<a href="mailto:roucaries.bastien@gmail.com">roucaries.bastien@gmail.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Le lun. 20 sept. 2021 à 17:28, Jérémy Lal <<a href="mailto:kapouer@melix.org" target="_blank">kapouer@melix.org</a>> a écrit :<br>
><br>
><br>
><br>
> Le lun. 20 sept. 2021 à 19:15, Ondrej Zary <<a href="mailto:ondrej@zary.sk" target="_blank">ondrej@zary.sk</a>> a écrit :<br>
> ><br>
> > On Monday 20 September 2021 16:56:18 Bastien ROUCARIES wrote:<br>
> > > Le lun. 20 sept. 2021 à 14:24, Ondrej Zary <<a href="mailto:ondrej@zary.sk" target="_blank">ondrej@zary.sk</a>> a écrit :<br>
> > > ><br>
> > > > On Monday 20 September 2021, Bastien ROUCARIES wrote:<br>
> > > > > Could you try to apply<br>
> > > > ><br>
> > > > > <a href="https://github.com/nodejs/node/commit/aa4611cccbcb197df51a9f7056d019005d91acf4" rel="noreferrer" target="_blank">https://github.com/nodejs/node/commit/aa4611cccbcb197df51a9f7056d019005d91acf4</a><br>
> > > > ><br>
> > > > > I think it describe that you see<br>
> > > ><br>
> > > > Does not apply, unfortunately. There's no node_dir.cc file and also no BaseObjectPtr definition.<br>
> > ><br>
> > > Ok as band aid could you replace in the patch BaseObjectPtr by<br>
> > > std:shared_ptr<ObjectPtr><br>
> ><br>
> > Biggest problem is the missing node_dir.cc file. The patched code from that file is not present at all in nodejs 10.<br>
><br>
> Wild guess, try only that part:<br>
><br>
> - delete wrap_;<br>
> + wrap_->Detach();<br>
> + wrap_.reset();<br>
Yes but reset is std:shared_ptr<ObjectPtr><br>
<br>
But I agree it is the main part of the patch<br></blockquote><div><br></div><div>It's interesting to compare to</div><div><a href="https://github.com/nodejs/node/commit/7f178663ebffc82c9f8a5a1b6bf2da0c263a30ed">https://github.com/nodejs/node/commit/7f178663ebffc82c9f8a5a1b6bf2da0c263a30ed</a></div><div><br></div><div>somewhat shows what's "allowed" to do.</div></div></div>