From a419fc54e634a28acd6b6e5ca6dbed764e48c244 Mon Sep 17 00:00:00 2001 From: Jon Trowbridge Date: Wed, 31 Oct 2001 06:44:00 +0000 Subject: Check that local_listener and other_contacts_listener are non-NULL before 2001-10-31 Jon Trowbridge * gui/component/select-names/e-select-names.c (e_select_names_destroy): Check that local_listener and other_contacts_listener are non-NULL before disconnecting/unrefing. * gui/component/select-names/e-select-names-manager.c (entry_destroyed): The entry shouldn't unref the manager. (e_select_names_manager_create_entry): The entry shouldn't hold a ref to the manager. It becomes circular. (e_select_names_manager_create_entry): On the other hand, we should hold a ref to the entry we create. * gui/component/select-names/e-select-names-bonobo.c (impl_destroy): Remove the explicit dialog destroy hack. svn path=/trunk/; revision=14519 --- .../gui/component/select-names/e-select-names-bonobo.c | 7 ------- .../gui/component/select-names/e-select-names-manager.c | 6 ++++-- addressbook/gui/component/select-names/e-select-names.c | 13 +++++++++---- 3 files changed, 13 insertions(+), 13 deletions(-) (limited to 'addressbook/gui') diff --git a/addressbook/gui/component/select-names/e-select-names-bonobo.c b/addressbook/gui/component/select-names/e-select-names-bonobo.c index 7f33d335b7..abe3af3634 100644 --- a/addressbook/gui/component/select-names/e-select-names-bonobo.c +++ b/addressbook/gui/component/select-names/e-select-names-bonobo.c @@ -356,13 +356,6 @@ impl_destroy (GtkObject *object) select_names = E_SELECT_NAMES_BONOBO (object); priv = select_names->priv; - - /* FIXME: a hack to work around some ref counting brokenness that - I'll fix later. */ - if (priv->manager->names) { - gtk_widget_destroy (GTK_WIDGET (priv->manager->names)); - priv->manager->names = NULL; - } gtk_object_unref (GTK_OBJECT (priv->manager)); diff --git a/addressbook/gui/component/select-names/e-select-names-manager.c b/addressbook/gui/component/select-names/e-select-names-manager.c index 2ba538b63f..a4b61def7e 100644 --- a/addressbook/gui/component/select-names/e-select-names-manager.c +++ b/addressbook/gui/component/select-names/e-select-names-manager.c @@ -394,7 +394,6 @@ entry_destroyed(EEntry *entry, ESelectNamesManager *manager) } } } - gtk_object_unref(GTK_OBJECT(manager)); } static void @@ -502,6 +501,8 @@ e_select_names_manager_create_entry (ESelectNamesManager *manager, const char *i entry->entry = eentry; entry->id = (char *)id; + gtk_object_ref (GTK_OBJECT (entry->entry)); + model = e_select_names_text_model_new (section->model); e_list_append (manager->entries, entry); g_free(entry); @@ -520,9 +521,10 @@ e_select_names_manager_create_entry (ESelectNamesManager *manager, const char *i "use_ellipsis", TRUE, "allow_newlines", FALSE, NULL); + gtk_signal_connect(GTK_OBJECT(eentry), "destroy", GTK_SIGNAL_FUNC(entry_destroyed), manager); - gtk_object_ref(GTK_OBJECT(manager)); + return GTK_WIDGET(eentry); } } diff --git a/addressbook/gui/component/select-names/e-select-names.c b/addressbook/gui/component/select-names/e-select-names.c index cb04e973a6..fe737bbce7 100644 --- a/addressbook/gui/component/select-names/e-select-names.c +++ b/addressbook/gui/component/select-names/e-select-names.c @@ -657,10 +657,15 @@ e_select_names_destroy (GtkObject *object) { ESelectNames *e_select_names = E_SELECT_NAMES(object); - gtk_signal_disconnect_by_data(GTK_OBJECT(e_select_names->local_listener), e_select_names); - gtk_object_unref(GTK_OBJECT(e_select_names->local_listener)); - gtk_signal_disconnect_by_data(GTK_OBJECT(e_select_names->other_contacts_listener), e_select_names); - gtk_object_unref(GTK_OBJECT(e_select_names->other_contacts_listener)); + if (e_select_names->local_listener) { + gtk_signal_disconnect_by_data(GTK_OBJECT(e_select_names->local_listener), e_select_names); + gtk_object_unref(GTK_OBJECT(e_select_names->local_listener)); + } + + if (e_select_names->other_contacts_listener) { + gtk_signal_disconnect_by_data(GTK_OBJECT(e_select_names->other_contacts_listener), e_select_names); + gtk_object_unref(GTK_OBJECT(e_select_names->other_contacts_listener)); + } gtk_object_unref(GTK_OBJECT(e_select_names->gui)); g_hash_table_foreach(e_select_names->children, (GHFunc) e_select_names_child_free, e_select_names); -- cgit v1.2.3