[PATCH] Take care also of the KJob::finished signal in KPart - it also fixes Ark's BUG 187538
alediaferia at gmail.com
Tue Apr 7 22:08:03 BST 2009
2009/4/7 Raphael Kubo da Costa <kubito at gmail.com>
> 2009/4/7 David Faure <faure at kde.org>:
> > On Tuesday 07 April 2009, David Faure wrote:
> >> I hadn't realized that the default for kill() had changed in kde4.
> > Ah!
> > It was changed, but in fact the other job trackers do call
> > ./kabstractwidgetjobtracker.cpp:80: job->kill(KJob::EmitResult); //
> notify that the job has been killed
> > ./kstatusbarjobtracker.cpp:143: job->kill(KJob::EmitResult); // notify
> that the job has been killed
> > which proves that the expected behavior hasn't really changed :)
> > The bug is only in kuiserverjobtracker.cpp which calls kill() without an
> > because of a direct signal/slot connection. I'm very much in favour of
> rakuco's patch
> > (with better handling of the race where the jobview isn't in the map
> > we're talking about it on irc now).
> I'm attaching the patch that's going to be committed. Q_ASSERT and
> keys() were dropped in favour of if (job) and key(). And sorry for the
> thread duplication ;)
The thread duplication is fault of mine ;-)
Finally we have a solution, yay!!
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the kde-core-devel