On Fri, Dec 17, 2010 at 4:12 AM, Olivier JG <olivier.jg@gmail.= com> wrote:

On 11/21/2010 01:00 AM, Milian Wolff wrot= e:

On Saturday 20 November 2010 17:50:31 Milian Wolff wrote:

Hey all,

currently the ImportProjectJob is not really interruptable. It has doKill that just waitForFinished which can easily lead to deadlocks nowadays,
since we can import stuff from the background bug QMetaType::invokeMethod in the projectmodel then. E.g.:

btw, I just notice that this is probably only hit when I try this patch:
diff --git a/shell/project.cpp b/shell/project.cpp
index c8fbc53..877c4a9 100644
--- a/shell/project.cpp
+++ b/shell/project.cpp
@@ -237,9 +237,10 @@ public:

=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0loading=3Dfalse;
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0if(job->errorText().isEmpty()) {
- =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0projCtrl->projectModel()->= appendRow(topItem);
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0projCtrl->projectImport= ingFinished( project );
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0} else {
+ =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0Q_ASSERT(topItem->index().isV= alid());
+ =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0projCtrl->projectModel()->= removeRow(topItem->row());
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0projCtrl->closeProject(= project);
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0}
=C2=A0 =C2=A0 =C2=A0}
@@ -516,6 +517,7 @@ bool Project::open( const KUrl& =C2=A0projectFileUr= l_ )
=C2=A0 =C2=A0 =C2=A0d->loadVersionControlPlugin(projectGroup);
=C2=A0 =C2=A0 =C2=A0d->progress->setBuzzy();
=C2=A0 =C2=A0 =C2=A0KJob* importJob =3D iface->createImportJob(d->to= pItem );
+ =C2=A0 =C2=A0Core::self()->projectController()->projectModel()->= appendRow(d->topItem);
=C2=A0 =C2=A0 =C2=A0connect( importJob, SIGNAL( result( KJob* ) ), this, S= LOT( importDone(
KJob* ) ) );
=C2=A0 =C2=A0 =C2=A0Core::self()->runController()->registerJob( impo= rtJob );
=C2=A0 =C2=A0 =C2=A0return true;

Which would be an (imo) awesome addition to KDevelop. It didn't used to= work
as items where added from random threads but now that they are always added=
from the UI it just works=E2=84=A2

As I noted here: https://bugs.kde.org/show_bug.cgi?id=3D256709 it seem= s that the ImportProjectJob cannot be canceled because it is run with QtCon= current::run(), which the documentation states doesn't support being ca= nceled. All you can do is wait for it to finish.
Until such a time as someone rewrites ProjectImportJob to support killing, = shouldn't the ImportProjectJob simply not advertise itself as Killable?=

(As a side note, when I remove the call to setCapabilities(killable), the s= top button still gets enabled, but simply doesn't do anything...)

-Olivier JG

--
KDevelop-devel mailing list
KDevelop-d= evel@kdevelop.org
https://barney.cs.uni-potsdam.de/mailman/listinfo/kdeve= lop-devel