<br><br><div class="gmail_quote">On Fri, Mar 15, 2013 at 8:34 AM, Simeon Bird <span dir="ltr"><<a href="mailto:bladud@gmail.com" target="_blank">bladud@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi,<br>
<br>
I read through the patches - 0002, 0004 and 0005 are good. For 0001,<br>
can you not instead of this:<br>
<br>
    lock.unlock();<br>
    QMutexLocker rmlock(&m_rm->mutex); // for updateKickOffLists, but<br>
must be locked first<br>
    lock.relock(); // we must respect the lock ordering!<br>
<br>
     const QString newNaoIdentifier =<br>
m_cache.value(NAO::identifier()).toString();<br>
     const QUrl newNieUrl = m_cache.value(NIE::url()).toUrl();<br>
     updateIdentifierLists( oldNaoIdentifier, newNaoIdentifier );<br>
<br>
do this:<br>
<br>
     const QString newNaoIdentifier =<br>
m_cache.value(NAO::identifier()).toString();<br>
     const QUrl newNieUrl = m_cache.value(NIE::url()).toUrl();<br>
<br>
    lock.unlock();<br>
    QMutexLocker rmlock(&m_rm->mutex); // for updateKickOffLists, but<br>
must be locked first<br>
<br>
     updateIdentifierLists( oldNaoIdentifier, newNaoIdentifier );<br>
<br>
No rule says you can't take the mutexes independently, only that if<br>
you take them both you must take the rmmutex first, no?<br>
<br>
For patch 3, how about the attached two patches instead?<br></blockquote><div><br>I like your patch about making init a readWrite one, but not about the resourceStart/stop one. I cannot comment on the issue above, I'll need to look at the code more carefully.<br>

<br>Also, I don't really understand your - ResourceData: Change updateKickOffLists to take three arguments, the<br>
uri, old and new values, patch. I think I'll try to look at it again, a little later.<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I pushed a feature branch: feature/resourcemanagercleanup with both<br>
these patch series in them. It's on top of KDE/4.10, and your patches<br>
are on top of mine. Does it make sense to you? Vishesh, what do you<br>
think?<br>
<br>
Thanks<br>
<span><font color="#888888">Simeon<br>
</font></span><br>_______________________________________________<br>
Nepomuk mailing list<br>
<a href="mailto:Nepomuk@kde.org" target="_blank">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>