[Github-comments] [geany] Proxy plugins (#629)

[Colomban Wendling]

> +			plugin_free((Plugin *) item->data);
> +		}
> +		item = next;
> +	}
> +}
> +
> +
> +/* Returns true if the removal was successful (=> never for non-proxies) */
> +static gboolean unregister_proxy(Plugin *proxy)
> +{
> +	gboolean is_proxy = FALSE;
> +	GList *node;
> +
> +	/* Remove the proxy from the proxy list first. It might appear more than once (once
> +	 * for each extension), but if it doesn't appear at all it's not actually a proxy */
> +	foreach_list_safe(node, active_proxies.head)

prev function does the exact same thing without a macro.  it would be more consistent using the same technique in the 2 places (either one is OK with me, @codebrainz would prefer previous func's one)

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