<p>So I found one more problem: we now substitute the anon_* names in the sidebar with the corresponding typedef name if it's something like typedef struct {} Foo; The problem is that the sorting in the tree is still done according to the anon_* name instead of the typedef name. I started working on making it based on the typedef name but things started getting ugly - e.g. in compare_symbol_lines() each of the scope components will have to be checked whether it's anon or not and substituted with the typedef name and the typedef_table has to be passed to many functions. Now I'm not sure if these complications are worth it - it was supposed to be a simple patch which doesn't complicate things too much.</p>

<p>There are several options what to do:</p>

<ol>
<li>Ignore the incorrect sorting and leave it as it is.</li>
<li>Remove the anon_* renaming patches completely.</li>
<li>Use the anon_* renaming only when sorting by line number (should be fine for any reasonable source).</li>
<li>Implement the fix (despite making things more complicated).</li>
</ol>

<p>Right now I'm leaning towards either (2) or (3). What do you think?</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/862#issuecomment-172344085">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ71s_sP2MZe1jhBNprjCMw5Men0Cks5pa7FQgaJpZM4HB6zU.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/862#issuecomment-172344085"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>