[Bug 24120] Refactor the McdDispatcher

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Sun Oct 18 18:50:15 CEST 2009


http://bugs.freedesktop.org/show_bug.cgi?id=24120





--- Comment #11 from Will Thompson <will.thompson at collabora.co.uk>  2009-10-18 09:50:13 PST ---
part3:

We should really add TpStringPool. It's silly to keep using the dynamic handle
repo and have to explain all over the place that they're not actually handles.

+  /* the ListNames call we'll make in _constructed is the initial lock */
+  self->priv->startup_lock = 1;

Wouldn't it be clearer to set that alongside the call?

Otherwise, looks good, in a kind of "well, it looks plausible, assuming the
rest of MC behaves how I expect it does" way. :)


-- 
Configure bugmail: http://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