X-Git-Url: http://git.maemo.org/git/?a=blobdiff_plain;f=src%2Fmodest-ui-dimming-manager.c;h=0beef151f6751976c53d7b1f036ab205ef1929e5;hb=ac5c3771b950ec505b72b8ef657a2c84df91e2cb;hp=3f33a2c80d11a3bbde15c1961ee1f11d68b74d7d;hpb=ee9a9871f25d1cbb863c76b57b5acadea4035695;p=modest diff --git a/src/modest-ui-dimming-manager.c b/src/modest-ui-dimming-manager.c index 3f33a2c..0beef15 100644 --- a/src/modest-ui-dimming-manager.c +++ b/src/modest-ui-dimming-manager.c @@ -105,6 +105,15 @@ modest_ui_dimming_manager_init (ModestUIDimmingManager *obj) } static void +remove_all_timeouts (gpointer key, + gpointer value, + gpointer user_data) +{ + if (GPOINTER_TO_INT (value) > 0) + g_source_remove (GPOINTER_TO_INT (value)); +} + +static void modest_ui_dimming_manager_finalize (GObject *obj) { ModestUIDimmingManagerPrivate *priv; @@ -114,8 +123,14 @@ modest_ui_dimming_manager_finalize (GObject *obj) if (priv->groups_map != NULL) g_hash_table_unref (priv->groups_map); - if (priv->delayed_calls != NULL) + if (priv->delayed_calls != NULL) { + /* Remove all pending calls */ + g_hash_table_foreach (priv->delayed_calls, + remove_all_timeouts, + NULL); + g_hash_table_unref (priv->delayed_calls); + } G_OBJECT_CLASS(parent_class)->finalize (obj); } @@ -187,13 +202,13 @@ process_dimming_rules_delayed (gpointer data) priv = MODEST_UI_DIMMING_MANAGER_GET_PRIVATE(helper->manager); timeout_handler = g_hash_table_lookup (priv->delayed_calls, helper->name); - MODEST_DEBUG_BLOCK(g_print ("---------------------HIT %d\n", GPOINTER_TO_INT (timeout_handler));); - if (GPOINTER_TO_INT (timeout_handler) > 0) { g_source_remove (GPOINTER_TO_INT (timeout_handler)); } + gdk_threads_enter (); modest_dimming_rules_group_execute (helper->group); + gdk_threads_leave (); return FALSE; } @@ -207,6 +222,7 @@ process_dimming_rules_delayed_destroyer (gpointer data) priv = MODEST_UI_DIMMING_MANAGER_GET_PRIVATE(helper->manager); g_hash_table_remove (priv->delayed_calls, helper->name); g_free (helper->name); + g_object_unref (helper->manager); g_slice_free (DelayedDimmingRules, helper); } @@ -233,9 +249,9 @@ modest_ui_dimming_manager_process_dimming_rules_group (ModestUIDimmingManager *s /* Create the helper and start the timeout */ helper = g_slice_new (DelayedDimmingRules); helper->group = group; - helper->manager = self; + helper->manager = g_object_ref (self); helper->name = g_strdup (group_name); - new_handler = g_timeout_add_full (G_PRIORITY_DEFAULT, 500, process_dimming_rules_delayed, + new_handler = g_timeout_add_full (G_PRIORITY_DEFAULT, 100, process_dimming_rules_delayed, helper, process_dimming_rules_delayed_destroyer); g_hash_table_insert (priv->delayed_calls, g_strdup (group_name), GINT_TO_POINTER (new_handler)); MODEST_DEBUG_BLOCK(g_print ("---------------------Adding %d\n", new_handler););