From kmail-devel  Sat Apr 28 13:44:50 2001
From: Denis Perchine <dyp () perchine ! com>
Date: Sat, 28 Apr 2001 13:44:50 +0000
To: kmail-devel
Subject: Re: Memory leaks in KMail
X-MARC-Message: https://marc.info/?l=kmail-devel&m=98846550621246

On Saturday 28 April 2001 19:03, Don Sanders wrote:
> On Saturday 28 April 2001 07:17, Denis Perchine wrote:
> > > > Read a CVS message, that memory leak was fixed in KHTML.
> > > > And thought that this will finally fix the problem with memory le=
ak
> > > > 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.
> >
> > Not completely... When I do this it still leaks memory. But amount is
> > less. Something like 40K per message. It looks like the leak is in
> > kdeui...
>
> I can't reproduce that. The only memory leak here is the one in
> updateMessageMenu.

I can confirm this. You are right. It grows only when I go to larger mess=
age.

> I tested this using the following method:
> 1) Read some messages.
> 2) Read those same messages again.
> 3) In top look at the SIZE of kmail
> 4) Read those same messages again about 100 times.
> 5) If in step 4 SIZE maxes out there is no memory leak when reading
> messages.
>
> Perhaps you were reading new messages every time, in which case memory
> grows when the new message selected is larger than any previous seen
> message.
>
> Or maybe another memory leak has been inserted in the last few days sin=
ce I
> updated.
>
> I agree it's a serious problem, if no one objects I'll fix the problem =
with
> updateMessageMenu, changing it back to the way it used to be if necessa=
ry.
>
> BFN,
> Don.
> _______________________________________________
> Kmail Developers mailing list
> Kmail@master.kde.org
> http://master.kde.org/mailman/listinfo/kmail

--=20
Sincerely Yours,
Denis Perchine

----------------------------------
E-Mail: dyp@perchine.com
HomePage: http://www.perchine.com/dyp/
FidoNet: 2:5000/120.5
----------------------------------
_______________________________________________
Kmail Developers mailing list
Kmail@master.kde.org
http://master.kde.org/mailman/listinfo/kmail