[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kmail-devel
Subject:    Re: Memory leaks in KMail
From:       Don Sanders <sanders () kde ! org>
Date:       2001-04-29 20:39:57
[Download RAW message or body]

On Sunday 29 April 2001 21:17, Stephan Kulow wrote:
> On Friday, 27. April 2001 22:46, Don Sanders wrote:
> > On Friday 27 April 2001 16:47, Denis Perchine wrote:
> > > Hello,
> > >
> > > Read a CVS message, that memory leak was fixed in KHTML.
> > > And thought that this will finally fix the problem with memory leak
> > > when reading mails. But it is not the case. When I go to the next
> > > message approximately 100 kbytes is leaked. At least KMail grows in
> > > size. Even when I came back to the same message it grows.
> > >
> > > Also it grows by approx. 200K when I right click on the item, and get
> > > popup menu. Possibly this is a problem of kdelibs... But this is really
> > > unacceptable. After a day of work KMail use 100-150Mb of memory.
> > >
> > > Any ideas?
> >
> > It's updateMessageMenu again. Just "return;" in
> > KMMainWin::updateMessageMenu() to verify that yourself.
> >
> > Maybe there was always a memory leak there, maybe it is much more of a
> > problem now as maybe updateMessageMenu is called much more often than it
> > used to be. Maybe it was only called on right click or when clicking on
> > the message menu but now it is (more or less) called whenever a message
> > is read.
> >
> > IMO obviously updateMessageMenu should only be called on right click of a
> > message header or when the message menu is pulled down. Coolo any chance
> > of you agreeing on this?
>
> As I told you, this can't work as the toolbar buttons are disabled too. The
> name is just wrong, haem historical.

I believe enabling/disabling of (not yet created?) toolbar buttons is less 
important than fixing the memory leak.

BFN,
Don.
_______________________________________________
Kmail Developers mailing list
Kmail@master.kde.org
http://master.kde.org/mailman/listinfo/kmail

[prev in list] [next in list] [prev in thread] [next in thread]