diff options
| author | David Laban <david.laban@collabora.co.uk> | 2011-05-23 15:53:19 -0400 |
|---|---|---|
| committer | David Laban <david.laban@collabora.co.uk> | 2011-05-24 11:07:27 -0400 |
| commit | 926c91aa8545460e04a4726762f93bb4a85451aa (patch) | |
| tree | 664114b5d49376b9f11e14a5058509808bae60e6 /telepathy-logger/text-event.c | |
| parent | 275e50a2774363161743a75f02d8d2e44f471373 (diff) | |
| download | telepathy-logger-926c91aa8545460e04a4726762f93bb4a85451aa.tar.gz | |
text-event: s/dup_supersedes/get_supersedes/
Copying and freeing lists of events is a waste of time.
Reviewed-by: Nicolas Dufresne <nicolas.dufresne@collabora.co.uk>
Diffstat (limited to 'telepathy-logger/text-event.c')
| -rw-r--r-- | telepathy-logger/text-event.c | 18 |
1 files changed, 5 insertions, 13 deletions
diff --git a/telepathy-logger/text-event.c b/telepathy-logger/text-event.c index f4f9c00..0047bf5 100644 --- a/telepathy-logger/text-event.c +++ b/telepathy-logger/text-event.c @@ -345,24 +345,16 @@ _tpl_text_event_add_supersedes (TplTextEvent *self, /** - * tpl_text_event_dup_supersedes + * tpl_text_event_get_supersedes * @self: a #TplTextEvent * - * Returns: (transfer full): A #GList of #TplTextEvent that this event - * supersedes. Should be freed using - * g_list_foreach (l, g_object_unref, NULL); g_list_free (l). + * Returns: (transfer none): A #GList of #TplTextEvent that this event + * supersedes. */ GList * -tpl_text_event_dup_supersedes (TplTextEvent *self) +tpl_text_event_get_supersedes (TplTextEvent *self) { - GList *supersedes = NULL; - GList *l; - - /* Iterate backwards to copy quickly (thanks GList) */ - for (l = self->priv->supersedes.tail; l != NULL; l = g_list_previous (l)) - supersedes = g_list_prepend (supersedes, g_object_ref (l->data)); - - return supersedes; + return self->priv->supersedes.head; } |