<blockquote>
<p>shouldn't the entry be removed if list==NULL like it used to do?</p>
</blockquote>

<p>Yeah, I guess it makes sense (I've updated the patch). On the other hand as the removal will cause re-balancing the tree so the potential performance gained by making the tree smaller will be offset by the rebalancing. We could also remove the item from the hash table if the whole tree is empty but hash table lookup is constant so we won't gain anything from that so it doesn't make sense to add extra code for that.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/geany/geany/pull/797#issuecomment-190222139">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ-U-mpoLx4Tdea9Bqec21jyuPgyBks5povQNgaJpZM4G0Ju3.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/pull/797#issuecomment-190222139"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>