<br><br><div class="gmail_quote">On Sun, Jul 8, 2012 at 4:30 PM, Volker Krause <span dir="ltr"><<a href="mailto:vkrause@kde.org" target="_blank">vkrause@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Sunday 08 July 2012 09:40:32 David Faure wrote:<br>
<div><div class="h5">> On Saturday 07 July 2012 11:13:08 Volker Krause wrote:<br>
> > On Friday 06 July 2012 14:44:30 David Faure wrote:<br>
> > > I found the reason for the kmail composer window being disabled forever,<br>
> > > in<br>
> > > some cases.<br>
> > ><br>
> > > A blocking nepomuk query from an akonadiserver thread never terminates,<br>
> > > because nepomukqueryservice crashes meanwhile.<br>
> > ><br>
> > > This leads to two bugs:<br>
> > ><br>
> > > 1) queryserviceclient should kill blocking queries when<br>
> > > _k_serviceUnregistered tells us the service crashed.<br>
> > > The attached patch works for me, OK if I commit it to all appropriate<br>
> > > branches? (kdelibs 4.8, kdelibs 4.9, nepomuk-core, akonadi master --<br>
> > > LOL)<br></div></div></blockquote><div><br>Ship it!<br> <br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div class="h5">

> ><br>
> > +1 as far as Akonadi is concerned. We don't have _k_serviceUnregistered at<br>
> > all though, so this might require backporting a few more changes of the<br>
> > queryserviceclient code.<br>
><br>
</div></div>> Yep, I have these commits ready for akonadi. Just waiting for an OK from the<br>
> nepomuk side.<br></blockquote><div><br>Why does Akonadi have its own copy of the QueryServiceClient? Can't you just use ours?<br><br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<br>
great<br>
<br>
> BTW, does search/nepomuksearchengine.cpp ever delete QueryServiceClient<br>
> instances? It seems to me that reloadSearches() leaks all the<br>
> already-running queries.<br>
<br>
reloadSearches() calls removeSearch() for all existing QueryServiceClients,<br>
which ends with a call to deleteLater(), so this looks ok to me.<br>
<br>
There is a theoretical scenario (existing query but no existing corresponding<br>
collection) where it would leak indeed, but that should never happen, so<br>
adding extra safety checks for that is probably not worth it. Or did you see<br>
any indication it's actually leaking?<br>
<br>
regards,<br>
Volker<br>_______________________________________________<br>
Nepomuk mailing list<br>
<a href="mailto:Nepomuk@kde.org">Nepomuk@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/nepomuk" target="_blank">https://mail.kde.org/mailman/listinfo/nepomuk</a><br>
<br></blockquote></div><br><br clear="all"><br>-- <br><span style="color:rgb(192,192,192)">Vishesh Handa</span><br><br>