[sugar] [PATCH] Push get_buddy from Connect into sugar.presence (#6473)
Guillaume Desmottes
guillaume.desmottes
Wed May 14 10:09:40 EDT 2008
Le mercredi 14 mai 2008 ? 15:58 +0200, Morgan Collett a ?crit :
> In case it is a channel with channel specific handles...
>
> if self.self_handle == cs_handle:
> # It's me, just get my global handle
> handle = self._conn.GetSelfHandle()
>
> We either need to call GetSelfHandle regardless of knowing whether the
> group has channel specific handles, or we need to call GetGroupFlags
> regardless of whether it is my handle. Is either of those less costly?
>
Right, I forgot self.self_handle could be channel specific too. My bad.
> > I'm wondering if it shouldn't be better to override watch_participants
> > (or add a new similar mechanism) returning directly buddies instead of
> > handles. Would it be possible/worth to completely hide handles and
> > always use buddies from the activity Pov?
>
> There's a chance that some will want to use telepathy calls directly
> in the activity, in case we end up oversimplifying the collaboration
> API. You could make the same argument about the bus names... but if
> you think it's a good idea, provide a patch we can review :)
I don't know really, was just a random thought. I guess your patch is
good for now. We'll still be able to improve this API later if needed.
G.
More information about the Sugar-devel
mailing list