1 files changed, 40 insertions, 23 deletions

diff --git a/src/fs/gnunet-fs-gtk_namespace_manager.c b/src/fs/gnunet-fs-gtk_namespace_manager.c
index 95763aef..69fe4969 100644
--- a/src/fs/gnunet-fs-gtk_namespace_manager.c
+++ b/src/fs/gnunet-fs-gtk_namespace_manager.c
@@ -50,15 +50,6 @@ struct GNUNET_GTK_NamespaceManagerContext
   struct GNUNET_GTK_MainWindowContext *main_ctx;
 };
 
-/**
- * THIS IS EVIL! Used to avoid opening more than one ns manager
- * at once. Could be fixed by saving a pointer to the ns manager window
- * somewhere in the main gnunet-fs-gtk window, but main window does
- * not have a context structure for such things at the moment, hence
- * this VERY EVIL GLOBAL VARIABLE!
- */
-static struct GNUNET_GTK_NamespaceManagerContext *ns_manager = NULL;
-
 #define GNUNET_GTK_KNOWN_NAMESPACES_IS_MINE_COLUMN 0
 #define GNUNET_GTK_KNOWN_NAMESPACES_RANK_COLUMN 1
 #define GNUNET_GTK_KNOWN_NAMESPACES_NAME_COLUMN 2
@@ -763,7 +754,6 @@ GNUNET_GTK_namespace_manager_dialog_response_cb (GtkDialog *dialog,
     free_lists_contents (ctx);
     gtk_widget_destroy (GTK_WIDGET (ctx->namespace_manager));
     GNUNET_free (ctx);
-    ns_manager = NULL;
   }
 }
 
@@ -898,17 +888,10 @@ known_ns_selection_changed (GtkTreeSelection *treeselection,
   gtk_widget_set_sensitive (ctx->details_apply_button, FALSE);
 }
 
-
-
-void
-GNUNET_GTK_main_menu_file_manage_pseudonyms_activate_cb (GtkMenuItem *menuitem,
-    gpointer user_data)
+GtkObject *
+GNUNET_GTK_namespace_manager_open (struct GNUNET_GTK_MainWindowContext *main_ctx)
 {
-  struct GNUNET_GTK_MainWindowContext *main_ctx = user_data;
-    struct GNUNET_GTK_NamespaceManagerContext *ctx;
-
-  if (ns_manager != NULL)
-    return;
+  struct GNUNET_GTK_NamespaceManagerContext *ctx;
 
   ctx = GNUNET_malloc (sizeof (struct GNUNET_GTK_NamespaceManagerContext));
   ctx->builder = GNUNET_GTK_get_new_builder ("gnunet_fs_gtk_namespace_manager.glade", ctx);
@@ -916,7 +899,7 @@ GNUNET_GTK_main_menu_file_manage_pseudonyms_activate_cb (GtkMenuItem *menuitem,
   {
     GNUNET_break (0);
     GNUNET_free (ctx);
-    return;
+    return NULL;
   }
 
   ctx->main_ctx = main_ctx;
@@ -967,12 +950,46 @@ GNUNET_GTK_main_menu_file_manage_pseudonyms_activate_cb (GtkMenuItem *menuitem,
   /* sort namespace order list by rank (initially) */
   sort_order_list (ctx, GNUNET_GTK_NAMESPACE_ORDER_RANK_COLUMN);
 
-  ns_manager = ctx;
-
   gtk_widget_set_sensitive (ctx->details_apply_button, FALSE);
 
   /* show dialog */
   gtk_window_present (GTK_WINDOW (ctx->namespace_manager));
+  return GTK_OBJECT (ctx->namespace_manager);
+}
+
+void
+ns_manager_deleted (GtkObject *object, gpointer user_data)
+{
+  struct GNUNET_GTK_MainWindowContext *ctx = user_data;
+  g_signal_handler_disconnect (object, ctx->ns_manager_delete_handler_id);
+  g_object_unref (G_OBJECT (ctx->ns_manager));
+  ctx->ns_manager = NULL;
+  ctx->ns_manager_delete_handler_id = 0;
+  return;
+}
+
+void
+GNUNET_GTK_open_ns_manager (struct GNUNET_GTK_MainWindowContext *ctx)
+{
+  if (ctx->ns_manager != NULL)
+  {
+    gtk_window_present (GTK_WINDOW (ctx->ns_manager));
+    return;
+  }
+
+  ctx->ns_manager = GNUNET_GTK_namespace_manager_open (ctx);
+  if (ctx->ns_manager == NULL)
+    return;
+
+  g_object_ref (G_OBJECT (ctx->ns_manager));
+  ctx->ns_manager_delete_handler_id = g_signal_connect (G_OBJECT (ctx->ns_manager), "destroy", G_CALLBACK (ns_manager_deleted), ctx);
+}
+
+void
+GNUNET_GTK_main_menu_file_manage_pseudonyms_activate_cb (GtkMenuItem *menuitem,
+    gpointer user_data)
+{
+  GNUNET_GTK_open_ns_manager (user_data);
 }
 
 /* end of gnunet-fs-gtk_namespace_manager.c */