[evolution-data-server] e_cal_backend_foreach_view: Use e_cal_backend_list_views().
- From: Matthew Barnes <mbarnes src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [evolution-data-server] e_cal_backend_foreach_view: Use e_cal_backend_list_views().
- Date: Sun, 7 Oct 2012 22:42:19 +0000 (UTC)
commit ca4417ce6dfef3c5f5814d081b13df749d965fae
Author: Matthew Barnes <mbarnes redhat com>
Date: Sun Oct 7 12:01:51 2012 -0400
e_cal_backend_foreach_view: Use e_cal_backend_list_views().
Holding a locked mutex while invoking callback functions we don't
control raises the potential for deadlock.
calendar/libedata-cal/e-cal-backend.c | 19 +++++++------------
1 files changed, 7 insertions(+), 12 deletions(-)
---
diff --git a/calendar/libedata-cal/e-cal-backend.c b/calendar/libedata-cal/e-cal-backend.c
index 1f78b35..a91abdd 100644
--- a/calendar/libedata-cal/e-cal-backend.c
+++ b/calendar/libedata-cal/e-cal-backend.c
@@ -814,24 +814,19 @@ e_cal_backend_foreach_view (ECalBackend *backend,
gpointer user_data),
gpointer user_data)
{
- GList *views;
- EDataCalView *view;
- gboolean stop = FALSE;
+ GList *list, *link;
- g_return_if_fail (backend != NULL);
+ g_return_if_fail (E_IS_CAL_BACKEND (backend));
g_return_if_fail (callback != NULL);
- g_mutex_lock (backend->priv->views_mutex);
+ list = e_cal_backend_list_views (backend);
- for (views = backend->priv->views; views && !stop; views = views->next) {
- view = E_DATA_CAL_VIEW (views->data);
-
- g_object_ref (view);
- stop = !callback (view, user_data);
- g_object_unref (view);
+ for (link = list; link != NULL; link = g_list_next (link)) {
+ if (!callback (E_DATA_CAL_VIEW (link->data), user_data))
+ break;
}
- g_mutex_unlock (backend->priv->views_mutex);
+ g_list_free_full (list, (GDestroyNotify) g_object_unref);
}
/**
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]