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

List:       kde-commits
Subject:    Re: kdebase/kfind
From:       Laurent Montel <montel () kde ! org>
Date:       2004-08-26 16:15:46
Message-ID: 200408261815.46710.montel () kde ! org
[Download RAW message or body]

On Thursday 26 August 2004 18:12, David Faure wrote:
> On Thursday 26 August 2004 18:08, Laurent Montel wrote:
> > CVS commit by mlaurent:
> >
> > Fix mem leak
> > (Backport ?)
> >
> >
> >   M +1 -0      kquery.cpp   1.35
> >
> >
> > --- kdebase/kfind/kquery.cpp  #1.34:1.35
> > @@ -29,4 +29,5 @@ KQuery::KQuery(QObject *parent, const ch
> >  KQuery::~KQuery()
> >  {
> > +                delete processLocate;
> >  }
>
> One could also do "new KProcess(this)" btw. Reduces the risk of double
> deletion etc.

it's right now KProcess have a QObject as attribute.
I will change it.
Regards.
[prev in list] [next in list] [prev in thread] [next in thread]