[Github-comments] [geany] Rewrite scope completion 3 (#862)

[Jiří Techet]

>  	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);

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.

---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/862/files#r49342461
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160111/cf06543a/attachment.html>