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 /tests | |
| 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 'tests')
| -rw-r--r-- | tests/dbus/test-tpl-log-store-xml.c | 16 |
1 files changed, 5 insertions, 11 deletions
diff --git a/tests/dbus/test-tpl-log-store-xml.c b/tests/dbus/test-tpl-log-store-xml.c index 89d5fe1..0e58536 100644 --- a/tests/dbus/test-tpl-log-store-xml.c +++ b/tests/dbus/test-tpl-log-store-xml.c @@ -538,13 +538,11 @@ test_add_superseding_event (XmlTestCaseFixture *fixture, assert_cmp_text_event (TPL_EVENT (new_event), events->data); /* Check that the two events are linked */ - superseded = tpl_text_event_dup_supersedes (events->data); + superseded = tpl_text_event_get_supersedes (events->data); g_assert (superseded != NULL); assert_cmp_text_event (event, superseded->data); - g_assert (tpl_text_event_dup_supersedes (superseded->data) == NULL); + g_assert (tpl_text_event_get_supersedes (superseded->data) == NULL); - g_list_foreach (superseded, (GFunc) g_object_unref, NULL); - g_list_free (superseded); g_list_foreach (events, (GFunc) g_object_unref, NULL); g_list_free (events); @@ -570,15 +568,13 @@ test_add_superseding_event (XmlTestCaseFixture *fixture, assert_cmp_text_event (TPL_EVENT (new_new_event), events->data); /* Check that the three events are linked */ - superseded = tpl_text_event_dup_supersedes (events->data); + superseded = tpl_text_event_get_supersedes (events->data); g_assert (superseded != NULL); assert_cmp_text_event (TPL_EVENT (new_event), superseded->data); g_assert (superseded->next != NULL); assert_cmp_text_event (event, superseded->next->data); - g_assert (tpl_text_event_dup_supersedes (superseded->next->data) == NULL); + g_assert (tpl_text_event_get_supersedes (superseded->next->data) == NULL); - g_list_foreach (superseded, (GFunc) g_object_unref, NULL); - g_list_free (superseded); g_list_foreach (events, (GFunc) g_object_unref, NULL); g_list_free (events); @@ -614,12 +610,10 @@ test_add_superseding_event (XmlTestCaseFixture *fixture, /* Check that the events are not linked (and a dummy was inserted instead) * because the timestamp was wrong. */ - superseded = tpl_text_event_dup_supersedes (events->data); + superseded = tpl_text_event_get_supersedes (events->data); g_assert (superseded != NULL); g_assert_cmpstr (tpl_text_event_get_message (superseded->data), ==, ""); - g_list_foreach (superseded, (GFunc) g_object_unref, NULL); - g_list_free (superseded); g_list_foreach (events, (GFunc) g_object_unref, NULL); g_list_free (events); |