X-Git-Url: http://pileus.org/git/?a=blobdiff_plain;f=src%2Fgis-plugin.c;h=ca009c98751b0955e25896d267feea018ea2e4ee;hb=8f2f3918e1bc04ac40a9ba0a24452c1b1b0183ac;hp=c6382380d5646d2f58ce2c43662c0fc2a1fe313d;hpb=0f8fb0a64c7b583ddfc8325e12ddf5475751bc86;p=grits

diff --git a/src/gis-plugin.c b/src/gis-plugin.c
index c638238..ca009c9 100644
--- a/src/gis-plugin.c
+++ b/src/gis-plugin.c
@@ -47,15 +47,10 @@ GType gis_plugin_get_type()
 	return type;
 }
 
-void gis_plugin_expose(GisPlugin *self)
-{
-	g_return_if_fail(GIS_IS_PLUGIN(self));
-	GIS_PLUGIN_GET_INTERFACE(self)->expose(self);
-}
-
 GtkWidget *gis_plugin_get_config(GisPlugin *self)
 {
-	g_return_val_if_fail(GIS_IS_PLUGIN(self), NULL);
+	if (!GIS_IS_PLUGIN(self))
+		return NULL;
 	GisPluginInterface *iface = GIS_PLUGIN_GET_INTERFACE(self);
 	return iface->get_config ? iface->get_config (self) : NULL;
 }
@@ -75,21 +70,22 @@ GisPlugins *gis_plugins_new(gchar *dir)
 	GisPlugins *plugins = g_new0(GisPlugins, 1);
 	if (dir)
 		plugins->dir = g_strdup(dir);
-	plugins->plugins = g_ptr_array_new();
 	return plugins;
 }
 
 void gis_plugins_free(GisPlugins *self)
 {
 	g_debug("GisPlugins: free");
-	for (int i = 0; i < self->plugins->len; i++) {
-		GisPluginStore *store = g_ptr_array_index(self->plugins, i);
+	for (GList *cur = self->plugins; cur; cur = cur->next) {
+		GisPluginStore *store = cur->data;
+		g_debug("GisPlugin: freeing %s refs=%d->%d", store->name,
+			G_OBJECT(store->plugin)->ref_count,
+			G_OBJECT(store->plugin)->ref_count-1);
 		g_object_unref(store->plugin);
 		g_free(store->name);
 		g_free(store);
-		g_ptr_array_remove_index(self->plugins, i);
 	}
-	g_ptr_array_free(self->plugins, TRUE);
+	g_list_free(self->plugins);
 	if (self->dir)
 		g_free(self->dir);
 	g_free(self);
@@ -119,7 +115,7 @@ GList *gis_plugins_available(GisPlugins *self)
 }
 
 GisPlugin *gis_plugins_load(GisPlugins *self, const char *name,
-		GisWorld *world, GisView *view, GisOpenGL *opengl, GisPrefs *prefs)
+		GisViewer *viewer, GisPrefs *prefs)
 {
 	g_debug("GisPlugins: load %s", name);
 	gchar *path = g_strdup_printf("%s/%s.%s", self->dir, name, G_MODULE_SUFFIX);
@@ -155,21 +151,21 @@ GisPlugin *gis_plugins_load(GisPlugins *self, const char *name,
 
 	GisPluginStore *store = g_new0(GisPluginStore, 1);
 	store->name = g_strdup(name);
-	store->plugin = constructor(world, view, opengl, prefs);
-	g_ptr_array_add(self->plugins, store);
+	store->plugin = constructor(viewer, prefs);
+	self->plugins = g_list_prepend(self->plugins, store);
 	return store->plugin;
 }
 
 gboolean gis_plugins_unload(GisPlugins *self, const char *name)
 {
 	g_debug("GisPlugins: unload %s", name);
-	for (int i = 0; i < self->plugins->len; i++) {
-		GisPluginStore *store = g_ptr_array_index(self->plugins, i);
+	for (GList *cur = self->plugins; cur; cur = cur->next) {
+		GisPluginStore *store = cur->data;
 		if (g_str_equal(store->name, name)) {
 			g_object_unref(store->plugin);
 			g_free(store->name);
 			g_free(store);
-			g_ptr_array_remove_index(self->plugins, i);
+			self->plugins = g_list_delete_link(self->plugins, cur);
 		}
 	}
 	return FALSE;
@@ -177,10 +173,12 @@ gboolean gis_plugins_unload(GisPlugins *self, const char *name)
 void gis_plugins_foreach(GisPlugins *self, GCallback _callback, gpointer user_data)
 {
 	g_debug("GisPlugins: foreach");
+	if (self == NULL)
+		return;
 	typedef void (*CBFunc)(GisPlugin *, const gchar *, gpointer);
 	CBFunc callback = (CBFunc)_callback;
-	for (int i = 0; i < self->plugins->len; i++) {
-		GisPluginStore *store = g_ptr_array_index(self->plugins, i);
+	for (GList *cur = self->plugins; cur; cur = cur->next) {
+		GisPluginStore *store = cur->data;
 		callback(store->plugin, store->name, user_data);
 	}
 }