<div dir="ltr">Hi,<div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 11, 2013 at 1:09 AM, Aleix Pol <span dir="ltr"><<a href="mailto:aleixpol@kde.org" target="_blank">aleixpol@kde.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Git commit 010d7bbb3f157faacdc65bab114d6e74c99afd08 by Aleix Pol.<br>
Committed on 10/09/2013 at 22:45.<br>
Pushed by apol into branch 'master'.<br>
<br>
Don't let the ProjectModel to be accessed from different threads<br>
<br>
Experience showed that it didn't work that well, especially since<br>
there's no aboutToChangeData().<br>@@ -199,7 +185,7 @@ ProjectBaseItem* ProjectBaseItem::takeRow(int row)<br>
     Q_ASSERT(row >= 0 && row < d->children.size());<br>
<br>
     if( model() ) {<br>
-        QMetaObject::invokeMethod( model(), "rowsAboutToBeRemoved", getConnectionTypeForSignalDelivery( model() ), Q_ARG(QModelIndex, index()), Q_ARG(int, row), Q_ARG(int, row) );<br>
+        QMetaObject::invokeMethod( model(), "rowsAboutToBeRemoved", Qt::DirectConnection, Q_ARG(QModelIndex, index()), Q_ARG(int, row), Q_ARG(int, row) );<br></blockquote><div><br></div><div>Shouldn't this be changed to a direct function call for easier understanding of the code (and all other cases too)?</div>
<div> </div><div>Andreas</div></div></div></div>