On 31 Mar 2001 22:19:40 +0200, Gediminas Paulauskas wrote:
> So, can I commit that? This changes quite a few files and keeping them
> up-to-date is not the task I want to do.

  OK, that's fine with me.  Please commit.

> I have looked into mail/folder-browser-factory.c. There should be the
> pixbuf cache, but what it currently does is load pixbufs to xml, and
> just keeping a copy of pixbuf, but not using it anymore (done_init=1).

  Hmm.  `done_init' is just for registering the pixbuf freeing function
with `atexit()' the first time.

  What happens is that it loads the pixbufs in the cache, and then
avoids loading them again the next time the component gets activated
again.  (`update_pixmaps()' is called from `control_activate()', which
in turn gets called every time you switch from another view to the
mailer view.)

> So I see no point in keeping it. I can make a general function to load
> pixmaps once, and put it into e-util/e-gui-utils. In c file have a
> GSList to track which static arrays of pixmap path/filename pairs have
> been initialized. Let me do a patch now.....

  The general function approach is what I would actually like to see.
:-)

-- 
Ettore

_______________________________________________
evolution-hackers maillist  -  [EMAIL PROTECTED]
http://lists.helixcode.com/mailman/listinfo/evolution-hackers

Reply via email to