<p><a href="https://github.com/kugel-" class="user-mention">@kugel-</a> Would you test this patch? I hope it could fix <a href="https://github.com/geany/geany-plugins/issues/604" class="issue-link js-issue-link" data-error-text="Failed to load issue title" data-id="260911239" data-permission-text="Issue title is private" data-url="https://github.com/geany/geany-plugins/issues/604">#604</a> - otherwise I don't know. Do you use some symlinks that could cause that the same path is re-visited many times? Or something that causes that tm_get_real_path() returns NULL for many files? These are the only two cases in which the directory wasn't properly closed (for normal projects I haven't been able to reproduce the issue - no dirs were open).</p>
<p>Note that if you had 1024 nested directories like</p>
<pre><code>first/second/third/.../1024-th
</code></pre>
<p>the current code will still need to open all the intermediate directories for traversal and you'll get the issue during project load. This would be rather hard to fix (or more precisely, the traversal would have to be slow because the intermediate dirs would have to be re-opened closed all the time) but this is a rather crazy case so I'll leave it as it is.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany-plugins/pull/605#issuecomment-332587745">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJxli5FgLEOmhtSxkmqyenHv6IDmVks5smn8SgaJpZM4PmFKv">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ0qOeyy8m5GAxbfQe_PBHYARItkSks5smn8SgaJpZM4PmFKv.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/geany/geany-plugins/pull/605#issuecomment-332587745"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/geany/geany-plugins","title":"geany/geany-plugins","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany-plugins"}},"updates":{"snippets":[{"icon":"PERSON","message":"@techee in #605: @kugel- Would you test this patch? I hope it could fix #604 - otherwise I don't know. Do you use some symlinks that could cause that the same path is re-visited many times? Or something that causes that tm_get_real_path() returns NULL for many files? These are the only two cases in which the directory wasn't properly closed (for normal projects I haven't been able to reproduce the issue - no dirs were open).\r\n\r\nNote that if you had 1024 nested directories like\r\n\r\n```\r\nfirst/second/third/.../1024-th\r\n```\r\n\r\nthe current code will still need to open all the intermediate directories for traversal and you'll get the issue during project load. This would be rather hard to fix (or more precisely, the traversal would have to be slow because the intermediate dirs would have to be re-opened closed all the time) but this is a rather crazy case so I'll leave it as it is."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany-plugins/pull/605#issuecomment-332587745"}}}</script>