[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