[Bug 54879] [next] initial 'next' branch

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Thu Sep 13 18:30:48 CEST 2012


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

--- Comment #4 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2012-09-13 16:30:48 UTC ---
In "replace now sealed struct member accessing with getters"

I think this could usefully be done on master. I'll prepare a patch based on
yours (I was actually about to do part of it for other reasons anyway).

> +    params = tp_protocol_borrow_params (protocol);

This function broke ABI between master and next, and I think Xavier might be
removing it altogether now. I think I'm going to just use dup_params() so the
same code works in both - efficiency is not exactly MC's strong point anyway.

> mcd_account_check_parameters

This still frees protocol with the TpConnectionManagerProtocol free-function.
That's not going to work. I see you fixed it in the next patch, though.

> check_one_parameter_update (McdAccount *account,
>                             TpConnectionManagerProtocol *protocol,

Similar. Again, the next patch fixes it.

> +    if (!tp_conn || tp_proxy_get_invalidated (TP_PROXY (tp_conn)) != NULL) return;

I'm not going to incorporate this patch without chopping off the "return" onto
its own line :-)

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



More information about the telepathy-bugs mailing list