<br><br><div class="gmail_quote">2009/4/7 Raphael Kubo da Costa <span dir="ltr"><<a href="mailto:kubito@gmail.com">kubito@gmail.com</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
2009/4/7 David Faure <<a href="mailto:faure@kde.org">faure@kde.org</a>>:<br>
<div class="im">> On Tuesday 07 April 2009, David Faure wrote:<br>
>> I hadn't realized that the default for kill() had changed in kde4.<br>
><br>
> Ah!<br>
> It was changed, but in fact the other job trackers do call kill(EmitResult):<br>
><br>
> ./kabstractwidgetjobtracker.cpp:80:    job->kill(KJob::EmitResult); // notify that the job has been killed<br>
> ./kstatusbarjobtracker.cpp:143:    job->kill(KJob::EmitResult); // notify that the job has been killed<br>
><br>
> which proves that the expected behavior hasn't really changed :)<br>
> The bug is only in kuiserverjobtracker.cpp which calls kill() without an EmitResult<br>
> because of a direct signal/slot connection. I'm very much in favour of rakuco's patch<br>
> (with better handling of the race where the jobview isn't in the map anymore,<br>
> we're talking about it on irc now).<br>
<br>
</div>I'm attaching the patch that's going to be committed. Q_ASSERT and<br>
keys() were dropped in favour of if (job) and key(). And sorry for the<br>
thread duplication ;)<br>
</blockquote></div><br>The thread duplication is fault of mine ;-)<br>Finally we have a solution, yay!!<br clear="all"><br>-- <br>Alessandro Diaferia<br>KDE Developer<br>