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

List:       kde-commits
Subject:    Re: KDE/kdelibs/kdecore/jobs (silent)
From:       Kevin Ottens <ervin () kde ! org>
Date:       2009-06-29 5:54:27
Message-ID: 200906290754.32458.ervin () kde ! org
[Download RAW message or body]


On Sunday 28 June 2009 16:36:33 David Faure wrote:
> On Sunday 28 June 2009, Ramon Zarazua wrote:
> > -     * @note: Deletes this job using deleteLater().
> > +     * @note: Deletes this job using deleteLater(), this function will
> > NOT return from +     *        your worker method.
> >       *
>
> What do you mean by worker method? And by "not return" ??
> Surely emitResult() returns. I don't understand this docu change.

Hmm, it indeed makes no sense to me. The other bit of change (swapping 
"delete" for "suicide") seems also wrong.

I'd propose to just revert this commit if we don't get some explanation about 
the intent of it.

Regards.
-- 
Kévin 'ervin' Ottens, http://ervin.ipsquad.net
"Ni le maître sans disciple, Ni le disciple sans maître,
Ne font reculer l'ignorance."

["signature.asc" (application/pgp-signature)]

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