From kde-commits Wed Jun 04 16:14:01 2008 From: "Eduardo Robles" Date: Wed, 04 Jun 2008 16:14:01 +0000 To: kde-commits Subject: Re: KDE/kdebase/apps/konqueror/src Message-Id: <9e2d3ad60806040914p64d392awee315d29c18e274 () mail ! gmail ! com> X-MARC-Message: https://marc.info/?l=kde-commits&m=121259730516519 MIME-Version: 1 Content-Type: multipart/mixed; boundary="------=_Part_2754_20130077.1212596041292" ------=_Part_2754_20130077.1212596041292 Content-Type: text/plain; charset=WINDOWS-1252 Content-Transfer-Encoding: quoted-printable Content-Disposition: inline On Wed, Jun 4, 2008 at 4:49 PM, Friedrich W. H. Kossebau wrote: > Am Mittwoch, 4. Juni 2008, um 15:38 Uhr, schrieb Luk=E1=9A Tinkl: > > Dne Sunday 1. of June 2008 01:34:12 Eduardo Robles Elvira napsal(a): > > > SVN commit 815027 by edulix: > > > > > > Fixing *huge* memory leak in konqueror: in KonqView::~KonqView() we > were > > > just not deleting the HistoryEntries. No kidding. Each view can conta= in > > > *a lot* of HistoryEntries, and each entry contains lots of data: url, > > > locationBarURL, title, buffer, postData, etc. All that was being leak= ed > > > when deleting konqviews (i.e. closing a tab). > > > > > > Thanks pinotree for helping out with valgrind and giving me the > valgrind > > > output, you rock! > > > > > > CCMAIL:kfm-devel@kde.org > > > CCMAIL:faure@kde.org > > > > Has this been backported? > > Even to KDE 3 perhaps? (I suspect for long all my dozens running konquero= r > instances to behave this way, so I have to restart once in a while instea= d > of > only suspend, to clear memory usage). Uhm no, it hasn't been backported, but probably it should. I will backport it ;-) ------=_Part_2754_20130077.1212596041292 Content-Type: text/html; charset=WINDOWS-1252 Content-Transfer-Encoding: quoted-printable Content-Disposition: inline On Wed, Jun 4, 2008 at 4:49 PM, Friedrich W. H. Kossebau <kossebau@kde.org> wrote:
Am Mittwoch, 4. Juni 2008, um 15:38 Uhr, schrieb Luk=E1=9A Tinkl:
> Dne Sunday 1. of June 2008 01:34= :12 Eduardo Robles Elvira napsal(a):
> > SVN commit 815027 by edulix:
> >
> > Fixing *huge* memory leak in konqueror: in KonqView::~KonqView() = we were
> > just not deleting the HistoryEntries. No kidding. Each view can c= ontain
> > *a lot* of HistoryEntries, and each entry contains lots of data: = url,
> > locationBarURL, title, buffer, postData, etc. All that was being = leaked
> > when deleting konqviews (i.e. closing a tab).
> >
> > Thanks pinotree for helping out with valgrind and giving me the v= algrind
> > output, you rock!
> >
> > CCMAIL:kfm-devel@kd= e.org
> > CCMAIL:faure@kde.org
>
> Has this been backported?

Even to KDE 3 perhaps? (I suspect for long all my dozens runnin= g konqueror
instances to behave this way, so I have to restart once in a while instead = of
only suspend, to clear memory usage).

Uhm no, it hasn't been backported, but probably it shoul= d. I will backport it ;-)

------=_Part_2754_20130077.1212596041292--