<p>In <a href="https://github.com/geany/geany/pull/862#discussion_r49342461">src/editor.c</a>:</p>
<pre style='color:#555'>>      GeanyFiletype *ft = editor->document->file_type;
> +  GPtrArray *tags;
> +  gboolean function = FALSE;
> +  gboolean member;
> +  gboolean ret = FALSE;
> +  const gchar *current_scope;
> +  const gchar *context_sep = symbols_get_context_separator(ft->id);
</pre>
<p>Yes, it's just a little less readable IMO. It's already used twice in the code below and there's the other extra "::" I forgot so. The three occurrences justify having it assigned into a variable IMO.</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/files#r49342461">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ5_FpBZqamBs3lbOcKnm_8Wh91Kiks5pY9EFgaJpZM4HB6zU.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/files#r49342461"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>