aboutsummaryrefslogtreecommitdiffstats
path: root/addressbook/gui
diff options
context:
space:
mode:
authorJon Trowbridge <trow@ximian.com>2001-09-15 14:05:29 +0800
committerJon Trowbridge <trow@src.gnome.org>2001-09-15 14:05:29 +0800
commit25d7fddd4ef5c143cf1746922ec331f3b2948954 (patch)
tree0f4980996e2214eb50c266fcaf315adc17017e79 /addressbook/gui
parent3328dab5cb971ebd87792b085d3d4fa36275ab18 (diff)
downloadgsoc2013-evolution-25d7fddd4ef5c143cf1746922ec331f3b2948954.tar
gsoc2013-evolution-25d7fddd4ef5c143cf1746922ec331f3b2948954.tar.gz
gsoc2013-evolution-25d7fddd4ef5c143cf1746922ec331f3b2948954.tar.bz2
gsoc2013-evolution-25d7fddd4ef5c143cf1746922ec331f3b2948954.tar.lz
gsoc2013-evolution-25d7fddd4ef5c143cf1746922ec331f3b2948954.tar.xz
gsoc2013-evolution-25d7fddd4ef5c143cf1746922ec331f3b2948954.tar.zst
gsoc2013-evolution-25d7fddd4ef5c143cf1746922ec331f3b2948954.zip
Make "stop" do nothing but set the stopped flag, as in
2001-09-15 Jon Trowbridge <trow@ximian.com> * backend/ebook/e-book-listener.c (e_book_listener_stop): Make "stop" do nothing but set the stopped flag, as in e-book-view-listener.c. (e_book_listener_destroy): Clean up our queue here, rather than in e_book_listener_stop. (response_free): Added. Move the rather lengthy bit of code needed to free a EBookListenerResponse into one place. (e_book_listener_check_queue): Properly deal with the stopped flag. (e_book_listener_queue_response): If the stopped flag is set, just drop the incoming response. * backend/ebook/e-book-view-listener.c (e_book_view_listener_stop): Make "stop" do nothing but set the stopped flag. (e_book_view_listener_destroy): Move all of the clean-up that used to be in e_book_view_listener_stop here. This considerably simplifies the logic required to avoid various race conditions. (e_book_view_listener_check_queue): Properly deal with the stopped flag. (e_book_view_listener_queue_response): Drop all incoming responses if we are stopped. 2001-09-14 Jon Trowbridge <trow@ximian.com> * backend/pas/pas-book.c (pas_book_queue_request): Hold a reference to the book on behalf of our idle function. (pas_book_check_queue): When we are finished, drop the reference we've been holding on behalf of the idle function. * backend/pas/pas-backend-file.c (pas_backend_file_process_create_card): Hold a reference to the book_view while sending our notifications. (pas_backend_file_process_remove_card): Hold a reference to the book_view while sending our notifications. * gui/contact-editor/e-contact-quick-add.c (quick_add_unref): Remove debugging spew. * backend/ebook/e-book-util.c: Remove a lot of unused code that worked around bugs that have long since been fixed. (simple_query_disconnect): Added. Breaks out the part of simple_query_free that disconnect signals. (simple_query_free): Replace code w/ a call to simple_query_disconnect. (simple_query_sequence_complete_cb): Call simple_query_disconnect before executing the callback, so that our callbacks don't get triggered by any book changes that might occur during that callback. * backend/ebook/e-book-view-listener.c (e_book_view_listener_check_queue): Changed to be consistent with e_book_listener_check_queue. (e_book_view_listener_queue_response): Also changed to use a high-frequency timeout. * backend/ebook/e-book-listener.c (e_book_listener_check_queue): OK, I've agonized over this stupid little function, and it should now be race-free. (e_book_listener_queue_response): We process our response queue in a high-frequency timeout rather than an idle. Using an idle function leads to some tricky race conditions and bad interactions with bonobo's semi-broken attempts to take over event processing. (e_book_view_listener_stop): Manually disable our timeout and clean up. svn path=/trunk/; revision=12851
Diffstat (limited to 'addressbook/gui')
-rw-r--r--addressbook/gui/contact-editor/e-contact-quick-add.c1
-rw-r--r--addressbook/gui/merging/e-card-merging.c4
2 files changed, 4 insertions, 1 deletions
diff --git a/addressbook/gui/contact-editor/e-contact-quick-add.c b/addressbook/gui/contact-editor/e-contact-quick-add.c
index 30c67a53ba..9559add2ee 100644
--- a/addressbook/gui/contact-editor/e-contact-quick-add.c
+++ b/addressbook/gui/contact-editor/e-contact-quick-add.c
@@ -83,7 +83,6 @@ quick_add_unref (QuickAdd *qa)
if (qa) {
--qa->refs;
if (qa->refs == 0) {
- g_message ("freeing %s / %s", qa->name, qa->email);
g_free (qa->name);
g_free (qa->email);
gtk_object_unref (GTK_OBJECT (qa->card));
diff --git a/addressbook/gui/merging/e-card-merging.c b/addressbook/gui/merging/e-card-merging.c
index d18c14f5bf..b9aacabde7 100644
--- a/addressbook/gui/merging/e-card-merging.c
+++ b/addressbook/gui/merging/e-card-merging.c
@@ -71,6 +71,7 @@ static void
match_query_callback (ECard *card, ECard *match, ECardMatchType type, gpointer closure)
{
ECardMergingLookup *lookup = closure;
+
if ((gint) type <= (gint) E_CARD_MATCH_VAGUE) {
doit (lookup);
g_free (lookup);
@@ -113,6 +114,7 @@ e_card_merging_book_add_card (EBook *book,
gpointer closure)
{
ECardMergingLookup *lookup;
+
lookup = g_new (ECardMergingLookup, 1);
lookup->op = E_CARD_MERGING_ADD;
@@ -122,6 +124,7 @@ e_card_merging_book_add_card (EBook *book,
lookup->closure = closure;
e_card_locate_match_full (book, card, NULL, match_query_callback, lookup);
+
return TRUE;
}
@@ -133,6 +136,7 @@ e_card_merging_book_commit_card (EBook *book,
{
ECardMergingLookup *lookup;
GList *avoid;
+
lookup = g_new (ECardMergingLookup, 1);
lookup->op = E_CARD_MERGING_COMMIT;