[Bug 77191] [next] simplify presence mixin: GHashTable -> "has message?"

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Wed Apr 9 13:26:23 PDT 2014


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

--- Comment #2 from Xavier Claessens <xclaesse at gmail.com> ---
+1 for the tp-glib commit 11c91762865545ab7d0704534ac7736d78795fdf. Didn't read
rest of that branch, neither did I read CM branches.

We forgot to _TP_SEAL() those presence mixin structs. TpPresenceStatusSpec has
all the getters needed to move the struct to presence-mixin.c but we should add
getters for TpPresenceStatus fields. Doing that saves us all that padding. I
did not grep CMs to know if that's possible to do at this point, though. That's
not preventing to merge your patch first though.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.


More information about the telepathy-bugs mailing list