[Dextrose] [PATCH] Simple messages notification extension

Bernie Innocenti bernie at codewiz.org
Thu Dec 2 11:38:52 EST 2010


On Thu, 2010-12-02 at 09:51 -0300, Martin Abente wrote:
>         
> 
> But in 0.88.1 users can't change the colors on the fly, right? The
> option in the CP requires restart.

Yes, but we'd like to remove this limitation one day.

More generally, caching values in global variables is premature
optimization which can lead to bugs. Even if profile.get_color() were
prohibitively slow, the caching optimization should be folded inside
that function rather than at every call sites.

-- 
   // Bernie Innocenti - http://codewiz.org/
 \X/  Sugar Labs       - http://sugarlabs.org/



More information about the Dextrose mailing list