<p>In <a href="https://github.com/geany/geany/pull/958#discussion_r66337142">src/symbols.c</a>:</p>
<pre style='color:#555'>> @@ -2071,102 +2071,114 @@ static TMTag *find_best_goto_tag(GeanyDocument *doc, GPtrArray *tags)
>  }
>  
>  
> +static GPtrArray *filter_tags(GPtrArray *tags, TMTag *current_tag, gboolean definition)
> +{
> +  const TMTagType forward_types = tm_tag_prototype_t | tm_tag_externvar_t;
> +  TMTag *tmtag, *last_tag = NULL;
> +  GPtrArray *filtered_tags = g_ptr_array_new();
> +  guint i;
> +
> +  foreach_ptr_array(tmtag, i, tags)
> +  {
> +          if (definition && !(tmtag->type & forward_types) ||
> +                  !definition && (tmtag->type & forward_types))
</pre>
<p>GCC suggests parentheses, and while the indention is fairly clear, I think it's still a good idea.</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/pull/958/files/65f84df5cae544deaf3a4ca56c7f29d8e13329d3#r66337142">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/ABDrJ4fZy4BDuK-3-cq9DZQ3KJr_CBUrks5qJysggaJpZM4HukG4">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ7z3lSiipSOxUnGQo578Fis8CfEiks5qJysggaJpZM4HukG4.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/958/files/65f84df5cae544deaf3a4ca56c7f29d8e13329d3#r66337142"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>