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

List:       kde-commits
Subject:    Re: KDE/kdebase/apps/konqueror/src
From:       "Eduardo Robles" <edulix () gmail ! com>
Date:       2008-06-04 16:14:01
Message-ID: 9e2d3ad60806040914p64d392awee315d29c18e274 () mail ! gmail ! com
[Download RAW message or body]

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 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%3Akfm-devel@kde.org>
> > > CCMAIL:faure@kde.org <CCMAIL%3Afaure@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 ;-)

[Attachment #3 (text/html)]

On Wed, Jun 4, 2008 at 4:49 PM, Friedrich W. H. Kossebau &lt;<a \
href="mailto:kossebau@kde.org">kossebau@kde.org</a>&gt; wrote:<br><div \
class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid \
rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> Am Mittwoch, 4. \
Juni 2008, um 15:38 Uhr, schrieb Lukáš Tinkl:<br> <div><div></div><div \
class="Wj3C7c">&gt; Dne Sunday 1. of June 2008 01:34:12 Eduardo Robles Elvira \
napsal(a):<br> &gt; &gt; SVN commit 815027 by edulix:<br>
&gt; &gt;<br>
&gt; &gt; Fixing *huge* memory leak in konqueror: in KonqView::~KonqView() we \
were<br> &gt; &gt; just not deleting the HistoryEntries. No kidding. Each view can \
contain<br> &gt; &gt; *a lot* of HistoryEntries, and each entry contains lots of \
data: url,<br> &gt; &gt; locationBarURL, title, buffer, postData, etc. All that was \
being leaked<br> &gt; &gt; when deleting konqviews (i.e. closing a tab).<br>
&gt; &gt;<br>
&gt; &gt; Thanks pinotree for helping out with valgrind and giving me the \
valgrind<br> &gt; &gt; output, you rock!<br>
&gt; &gt;<br>
&gt; &gt; <a href="mailto:CCMAIL%3Akfm-devel@kde.org">CCMAIL:kfm-devel@kde.org</a><br>
 &gt; &gt; <a href="mailto:CCMAIL%3Afaure@kde.org">CCMAIL:faure@kde.org</a><br>
&gt;<br>
&gt; Has this been backported?<br>
<br>
</div></div>Even to KDE 3 perhaps? (I suspect for long all my dozens running \
konqueror<br> instances to behave this way, so I have to restart once in a while \
instead of<br> only suspend, to clear memory usage).<font \
color="#888888"></font></blockquote><div><br>Uhm no, it hasn&#39;t been backported, \
but probably it should. I will backport it ;-)<br></div></div><br>



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