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

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


Sorry, this is for the view-size branch.

Ar 11/08/2008 am 17:49, ysgrifennodd Dafydd Harries:
> 
> >+    def remove_buddy(self, buddy):
> >+        self.results.remove(buddy)
> >+        self.handler.buddy_removed(buddy)
> 
> Perhaps we can add a FIXME here; something like:
> 
>   # FIXME: add more results here if len(self.results) <= self.size
> 
> Otherwise, +1.
> 
> -- 
> Dafydd
> _______________________________________________
> Code-review mailing list
> Code-review at lists.laptop.org
> http://lists.laptop.org/listinfo/code-review
> 

-- 
Dafydd


More information about the Code-review mailing list