Excerpts from Simon Schampijer's message of Thu Mar 24 21:16:32 +0100 2011:

> The patch does cache the palette of the Owner, this has been
> the behavior before 356641c332d6cc889b265dfc29598004cf37763c. As
> registering (and therefore the change of the palette) happens
> rather seldom compared to the change of a buddy palette, I think
> the Owner palette can be considered to be mostly static
> and therefore we can cache it.

I'd approach this from a different angle (than the palette being static)
and prefer the second patch [1] you attached to #2726 [2]: The owner
palette provides access to functions (My Settings, Register, end
session) that I don't expect to be used rather often during the lifetime
of a Sugar session and the palette can easily be reconstructed, so we
don't need to cache it (thus saving a tiny bit of memory).

You can consider this an Ack for your second patch. I trust you to add a
nice description. ;)

BTW: Thanks for all your recent patches!

Sascha

[1] https://bugs.sugarlabs.org/raw-attachment/ticket/2726/no_cache.patch
[2] https://bugs.sugarlabs.org/ticket/2726
-- 
http://sascha.silbe.org/
http://www.infra-silbe.de/

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Sugar-devel mailing list
Sugar-devel@lists.sugarlabs.org
http://lists.sugarlabs.org/listinfo/sugar-devel

Reply via email to