[Code-review] review: Gadget current-activity branch

Dafydd Harries dafydd.harries at collabora.co.uk
Mon Aug 11 12:13:41 EDT 2008


>+        # (room, activity id)
>+        self.current_activity = ('', '')

Why not make this an activity object? (Or None for no current activity.)

>+        # FIXME: what about buddies who are in activity views as
>+        # participants ?

I don't understand this comment. Could you elaborate?

-- 
Dafydd


More information about the Code-review mailing list