<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Em sex, 8 de fev de 2019 às 07:58, Roman Haefeli <<a href="mailto:reduzent@gmail.com">reduzent@gmail.com</a>> escreveu:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Alex<br>
<br>
I see that your version of [conv~] uses [else/dir] internally and this<br>
external of yours is still leaking file handles. I think I already<br>
reported this, but can't find an open issue about it. Maybe you closed<br>
them, despite them not being fixed.<br></blockquote><div><br></div><div>nope, still open, and yeah, we already tried to fix that but it wasn't fully solved :/ </div><div><br></div><div>thanks all for clarifying this. I guess I should get back to trying to fix it ;) </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Anyway, last time I checked, each instance of [dir] was not leaking<br>
three file handles anymore, but only one. So there is an improvement,<br>
but the issue is not fixed. You really need to close all file handles<br>
that you opened before.<br></blockquote><div><br></div><div>Yeah, that was the best I could do so far...</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Roman<br>
<br>
P.S.: I'm kinda happy to see that it bites you. This may sound mean,<br>
but I do think that else is a great library and thus the more danger is<br>
in such issues not being fixed. So: Thanks for the library, thanks for<br>
fixing the issue!</blockquote><div><br></div><div>I'm actually glad too, haha... but with mixed feelings. I wasn't actually aware this could lead to potential issues.</div><div><br></div><div>cheers </div></div></div>