[Bug 31668] break C API to drop deprecated stuff

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Thu Jul 12 16:05:23 CEST 2012


https://bugs.freedesktop.org/show_bug.cgi?id=31668

--- Comment #21 from Jonny Lamb <jonny.lamb at collabora.co.uk> 2012-07-12 14:05:23 UTC ---
Xavier merged master into next and removed some deprecations here:

http://cgit.collabora.com/git/user/xclaesse/telepathy-glib.git/log/?h=next-remove-deprecated

Some comments:

02050c89a381c9e9e:

You could use tp_asv_new (NULL, NULL) instead of g_hash_table_new_full (blah
blah blah).

cdb02889684c082:

+ test->sms_channel = TP_TEXT_CHANNEL (tp_client_factory_ensure_channel (
+     factory, test->connection, chan_path, props, &test->error));
  g_assert_no_error (test->error);
+ g_assert (TP_IS_TEXT_CHANNEL (test->channel));

The second assertion should be test->sms_channel, no? It's also redundant
(there's another too) as the TP_TEXT_CHANNEL cast will typecheck, no?

Otherwise, seems good.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
You are the assignee for the bug.



More information about the telepathy-bugs mailing list