<br><br><div class="gmail_quote">2009/4/6 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;">
<div class="im">2009/4/6 Alessandro Diaferia <<a href="mailto:alediaferia@gmail.com">alediaferia@gmail.com</a>>:<br>
</div><div class="im">> And so, couldn't we make kill() emitting result() ?<br>
</div>I don't know whether the current behaviour in KJob is wrong or not -<br>
KJob::kill() does emit the result signal, but only when called with<br>
KJob::EmitResult instead of KJob::Quietly. If you make kill() emit<br>
result every time, there's no point in making the distinction between<br>
KJob::EmitResult and KJob::Quietly.<br>
<div class="im"><br>
> Or, at least apidox should be updated and somehow connect to the job deletion in the ReadOnlyPart class..<br>
</div>There's another solution proposed by Aaron Seigo that would be<br>
creating another slot in KJob that would call kill(EmitResult), but it<br>
sounds a little hackish too.<br>
<br>
I think we first need to settle on where the problem actually lies -<br>
is it in KJob::kill's behaviour of not always emitting both finished<br>
and result, or is it in KUiServerJobTracker's way of connecting its<br>
stop button's clicked() signal, or even the signal connection in<br>
ReadOnlyPart?<br>
</blockquote></div><br clear="all">For me the default behavior should be KJob::EmitResult and not KJob::Quietly.. but those are just my 2 cents :)<br><br>-- <br>Alessandro Diaferia<br>KDE Developer<br>