@b4n commented on this pull request.


In src/pluginextension.c:

> +	const PluginExtensionEntry *entry_b = b;
+
+	return entry_b->priority - entry_a->priority;
+}
+
+
+GEANY_API_SYMBOL
+void plugin_extension_register(PluginExtension *extension, const gchar *plugin_name,
+	gint priority, gpointer data)
+{
+	PluginExtensionEntry *entry = g_malloc(sizeof *entry);
+
+	entry->extension = extension;
+	entry->data = data;
+	entry->priority = priority;
+	entry->plugin_name = g_strdup(plugin_name ? plugin_name : "unnamed plugin");

any reason to allow a NULL name? (and if so, not to leave it NULL internally, at least for now?)


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <geany/geany/pull/3849/review/2114155727@github.com>