> @@ -1028,6 +1298,17 @@ void plugins_init(void)
>
> g_signal_connect(geany_object, "save-settings", G_CALLBACK(update_active_plugins_pref), NULL);
> stash_group_add_string_vector(group, &active_plugins_pref, "active_plugins", NULL);
> +
> + g_queue_push_head(&active_proxies, &builtin_so_proxy);
> +}
> +
> +
> +/* Same as plugin_free(), except it does nothing for pluxies-in-use, to be called on
s/pluxies/proxies/
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/629/files#r40363830
> fname = g_build_filename(path, item->data, NULL);
> - if (plugin_new(fname, FALSE, TRUE))
> - count++;
> + proxy = is_plugin(fname);
> + if (proxy != NULL)
> + {
> + if (plugin_new(proxy->plugin, fname, FALSE, TRUE))
those two tests could be combined in a single `if()` (no biggie)
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/629/files#r40363668
> @@ -858,18 +1098,19 @@ load_plugins_from_path(const gchar *path)
> GSList *list, *item;
> gchar *fname, *tmp;
while at it, moving the declaration of `fname` in the inner scope would be perfect :)
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/629/files#r40363513
> {
> - const gchar *fname = active_plugins_pref[i];
> -
> - if (!EMPTY(fname) && g_file_test(fname, G_FILE_TEST_EXISTS))
> + proxies = active_proxies.length;
> + if (failed_plugins_list != NULL)
test is unnecessary as `NULL` is a valid GList
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/629/files#r40362717