[Bug 49372] deprecate tp_account/connection/channel_new
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Tue Jul 3 13:30:56 CEST 2012
https://bugs.freedesktop.org/show_bug.cgi?id=49372
--- Comment #15 from Xavier Claessens <xclaesse at gmail.com> 2012-07-03 04:30:56 PDT ---
(In reply to comment #12)
> Comment on attachment 61957 [details] [review]
> Examples: stop using tp_account/connection/channel_new()
>
> Review of attachment 61957 [details] [review]:
> -----------------------------------------------------------------
>
> ::: 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.
True, lots of examples are/contains not really exemplary code. I reported bug
#51687 for that, let's consider it out of scope for this bug.
> @@ +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?
Good catch.
--
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