[Bug 46358] In python environment, list_connection_managers passes an empty list as cms.

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Wed Feb 22 20:46:42 CET 2012


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

Simon McVittie <simon.mcvittie at collabora.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|telepathy-bugs at lists.freede |simon.mcvittie at collabora.co
                   |sktop.org                   |.uk
             Blocks|                            |31668
                URL|http://telepathy.freedeskto |
                   |p.org/doc/telepathy-glib/te |
                   |lepathy-glib-connection-man |
                   |ager.html#TpConnectionManag |
                   |erListCb                    |

--- Comment #1 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2012-02-22 11:46:42 PST ---
(In reply to comment #0)
> n_cms is passed with the correct value, however cms is passed as an empty list.

I think the right solution for this is to add a
tp_list_connection_managers_async()/tp_list_connection_managers_finish() pair
which is nicely introspectable and does sensible things (use GAsyncResult,
etc.), and eventually deprecate/remove tp_list_connection_managers.

(Also, we can take this opportunity to make it always-async, i.e. guarantee to
call the callback from the main loop, and never be re-entrant.)

I've written a simple tp_list_connection_managers_async() in terms of
tp_list_connection_managers(), but they should probably have common code rather
than one calling the other.

Also, we need to decide what _finish() returns in C - GList? GPtrArray?
(transfer container)? (transfer full)? See Bug #39189.

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