diff options
| author | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2014-03-26 18:40:12 +0000 |
|---|---|---|
| committer | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2014-03-27 15:42:31 +0000 |
| commit | 49e67694087d4841dfb400b901e6bb1dbde49608 (patch) | |
| tree | baa333bb3b10974d6366b7fc0c2afa24ae6598fb | |
| parent | b42a50b8aa3f971744f884834373463616032eb9 (diff) | |
| download | telepathy-glib-49e67694087d4841dfb400b901e6bb1dbde49608.tar.gz | |
tp_dbus_daemon_try_register_object: avoid GSList
The space optimization is hardly relevant here.
"I think GSList's only purpose is making your app crash if you use
g_list_free on it" -xclaesse
| -rw-r--r-- | telepathy-glib/dbus-daemon.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/telepathy-glib/dbus-daemon.c b/telepathy-glib/dbus-daemon.c index 3602ca88a..68be4bf32 100644 --- a/telepathy-glib/dbus-daemon.c +++ b/telepathy-glib/dbus-daemon.c @@ -371,7 +371,7 @@ struct _Registration { /* (transfer full) */ gchar *object_path; /* (transfer full) */ - GSList *skeletons; + GList *skeletons; }; static GQuark @@ -391,7 +391,7 @@ static void tp_dbus_daemon_registration_free (gpointer p) { Registration *r = p; - GSList *iter; + GList *iter; DEBUG ("%s (r=%p)", r->object_path, r); @@ -403,7 +403,7 @@ tp_dbus_daemon_registration_free (gpointer p) g_object_unref (iter->data); } - g_slist_free (r->skeletons); + g_list_free (r->skeletons); g_free (r->object_path); g_clear_object (&r->conn); g_slice_free (Registration, r); @@ -565,7 +565,7 @@ tp_dbus_daemon_try_register_object (TpDBusDaemon *self, goto finally; } - r->skeletons = g_slist_prepend (r->skeletons, skeleton); + r->skeletons = g_list_prepend (r->skeletons, skeleton); DEBUG ("- %s skeleton %p (wrapping %s %p)", iinfo->interface_info->name, skeleton, g_type_name (iface), object); |