As done with my patch, in the last mail i sent to the mailing list (sorry for duplicating) just connecting to the finished() signal solves the issue.. But i don't really know whether my approach is correct or not.<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"><br>
<br>
> On 2009-04-05 12:37:46, Aaron Seigo wrote:<br>
> > shouldn't the job emit the result signal before being autodeleted in this case? seems like a bug in kio to me ...<br>
<br>
</div>That's true, I've come to the conclusion that this patch is actually pretty hackish.<br>
<br>
Actually, the problem is in kdelibs/kdeui/kuiserverjobtracker.cpp (lines 84-85) -- it's connecting the signal emitted when the Stop button in the job tracker is pressed to a call to KJob::kill without passing KJob::EmitResult as an argument. By default, the argument is KJob::Quietly, which does not emit the result signal.<br>

<br>
However, I don't know how it can be fixed with the current structure -- kdelibs/kdeui/kstatusbarjobtracker.cpp implements a killJob method, but KUiServerJobTracker doesn't store the job as a member.<br>
<font color="#888888"><br>
<br>
- Raphael<br>
</font><div class="im"><br>
<br>
-----------------------------------------------------------<br>
This is an automatically generated e-mail. To reply, visit:<br>
<a href="http://reviewboard.kde.org/r/524/#review830" target="_blank">http://reviewboard.kde.org/r/524/#review830</a><br>
-----------------------------------------------------------<br>
<br>
<br>
</div><div><div></div><div class="h5">On 2009-04-04 18:57:16, Raphael Kubo da Costa wrote:<br>
><br>
> -----------------------------------------------------------<br>
> This is an automatically generated e-mail. To reply, visit:<br>
> <a href="http://reviewboard.kde.org/r/524/" target="_blank">http://reviewboard.kde.org/r/524/</a><br>
> -----------------------------------------------------------<br>
><br>
> (Updated 2009-04-04 18:57:16)<br>
><br>
><br>
> Review request for kdelibs.<br>
><br>
><br>
> Summary<br>
> -------<br>
><br>
> As the listed bugs say, when a URL is passed to a program and its ReadOnlyPart opens it, the program crashes when closed if the transfer is stoppped before it's finished. Calling "ark www.foo.bar/baz.zip" or "okular www.foo.bar/baz.pdf" both present this behaviour.<br>

><br>
> This patch makes the registered FileCopyJob not autodelete itself when the transfer is stopped, thus making ReadOnlyPart::abortLoad not access a dead object.<br>
><br>
><br>
> This addresses bugs 186722, 186980, 187538, and 188231.<br>
>     <a href="https://bugs.kde.org/show_bug.cgi?id=186722" target="_blank">https://bugs.kde.org/show_bug.cgi?id=186722</a><br>
>     <a href="https://bugs.kde.org/show_bug.cgi?id=186980" target="_blank">https://bugs.kde.org/show_bug.cgi?id=186980</a><br>
>     <a href="https://bugs.kde.org/show_bug.cgi?id=187538" target="_blank">https://bugs.kde.org/show_bug.cgi?id=187538</a><br>
>     <a href="https://bugs.kde.org/show_bug.cgi?id=188231" target="_blank">https://bugs.kde.org/show_bug.cgi?id=188231</a><br>
><br>
><br>
> Diffs<br>
> -----<br>
><br>
>   /trunk/KDE/kdelibs/kparts/part.cpp 948889<br>
><br>
> Diff: <a href="http://reviewboard.kde.org/r/524/diff" target="_blank">http://reviewboard.kde.org/r/524/diff</a><br>
><br>
><br>
> Testing<br>
> -------<br>
><br>
> Both Ark and Okular stopped crashing here.<br>
><br>
><br>
> Thanks,<br>
><br>
> Raphael<br>
><br>
><br>
<br>
</div></div></blockquote></div><br>Cheers<br clear="all"><br>-- <br>Alessandro Diaferia<br>KDE Developer<br>