[Bug 49372] deprecate tp_account/connection/channel_new

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Mon Jul 2 14:27:04 CEST 2012


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

--- Comment #12 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2012-07-02 05:27:04 PDT ---
Comment on attachment 61957
  --> https://bugs.freedesktop.org/attachment.cgi?id=61957
Examples: stop using tp_account/connection/channel_new()

Review of attachment 61957:
 --> (https://bugs.freedesktop.org/page.cgi?id=splinter.html&bug=49372&attachment=61957)
-----------------------------------------------------------------

::: examples/client/extended-client.c
@@ +211,5 @@
>  
>    if (die_if (error, "RequestConnection()"))
>      return;
>  
>    /* FIXME: there should be convenience API for this */

Is this still applicable? Probably not, because real applications (except MC)
shouldn't be using RequestConnection anyway.

@@ +212,5 @@
>    if (die_if (error, "RequestConnection()"))
>      return;
>  
>    /* FIXME: there should be convenience API for this */
> +  factory = tp_simple_client_factory_new (NULL);

Isn't this leaked?

-- 
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