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/
signature.asc
Description: PGP signature
_______________________________________________ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel