Don't crash if we don't have a terminal

[dolphin.git] / src / views / versioncontrol / updateitemstatesthread.cpp

diff --git a/src/views/versioncontrol/updateitemstatesthread.cpp b/src/views/versioncontrol/updateitemstatesthread.cpp
index e07d72c76d380fb6dc1b112a3d2e95c1593e60ad..0a183b4a58e1391978660f892890307b5823c86e 100644 (file)
--- a/src/views/versioncontrol/updateitemstatesthread.cpp
+++ b/src/views/versioncontrol/updateitemstatesthread.cpp
@@ -19,17 +19,13 @@
 
 #include "updateitemstatesthread.h"
 
 
 #include "updateitemstatesthread.h"
 

-#include <kversioncontrolplugin2.h>

 
 

-#include <QMutexLocker>
-
-UpdateItemStatesThread::UpdateItemStatesThread() :
+UpdateItemStatesThread::UpdateItemStatesThread(KVersionControlPlugin* plugin,
+                                               const QMap<QString, QVector<VersionControlObserver::ItemState> >& itemStates) :

     QThread(),
     QThread(),

-    m_globalPluginMutex(0),
-    m_plugin(0),
-    m_itemMutex(),
-    m_retrievedItems(false),
-    m_itemStates()
+    m_globalPluginMutex(nullptr),
+    m_plugin(plugin),
+    m_itemStates(itemStates)

 {
     // Several threads may share one instance of a plugin. A global
     // mutex is required to serialize the retrieval of version control
 {
     // Several threads may share one instance of a plugin. A global
     // mutex is required to serialize the retrieval of version control


@@ -42,71 +38,30 @@ UpdateItemStatesThread::~UpdateItemStatesThread()
 {
 }
 
 {
 }
 

-void UpdateItemStatesThread::setData(KVersionControlPlugin* plugin,
-                                     const QList<VersionControlObserver::ItemState>& itemStates)
-{
-    // The locks are taken in the same order as in run()
-    // to avoid potential deadlock.
-    QMutexLocker pluginLocker(m_globalPluginMutex);
-    QMutexLocker itemLocker(&m_itemMutex);
-
-    m_itemStates = itemStates;
-    m_plugin = plugin;
-}
-

 void UpdateItemStatesThread::run()
 {
     Q_ASSERT(!m_itemStates.isEmpty());
     Q_ASSERT(m_plugin);
 
 void UpdateItemStatesThread::run()
 {
     Q_ASSERT(!m_itemStates.isEmpty());
     Q_ASSERT(m_plugin);
 

-    QMutexLocker itemLocker(&m_itemMutex);
-
-    const QString directory = m_itemStates.first().item.url().directory(KUrl::AppendTrailingSlash);
-    m_retrievedItems = false;
-    itemLocker.unlock();
-

     QMutexLocker pluginLocker(m_globalPluginMutex);
     QMutexLocker pluginLocker(m_globalPluginMutex);

-    if (m_plugin->beginRetrieval(directory)) {
-        itemLocker.relock();
-        const int count = m_itemStates.count();
-
-        KVersionControlPlugin2* pluginV2 = qobject_cast<KVersionControlPlugin2*>(m_plugin);
-        if (pluginV2) {
-            for (int i = 0; i < count; ++i) {
-                m_itemStates[i].version = pluginV2->itemVersion(m_itemStates[i].item);
-            }
-        } else {
+    QMap<QString, QVector<VersionControlObserver::ItemState> >::iterator it = m_itemStates.begin();
+    for (; it != m_itemStates.end(); ++it) {
+        if (m_plugin->beginRetrieval(it.key())) {
+            QVector<VersionControlObserver::ItemState>& items = it.value();
+            const int count = items.count();

             for (int i = 0; i < count; ++i) {
             for (int i = 0; i < count; ++i) {

-                const KVersionControlPlugin::VersionState state = m_plugin->versionState(m_itemStates[i].item);
-                m_itemStates[i].version = static_cast<KVersionControlPlugin2::ItemVersion>(state);
+                const KFileItem& item = items.at(i).first;
+                const KVersionControlPlugin::ItemVersion version = m_plugin->itemVersion(item);
+                items[i].second = version;

             }
         }
 
         m_plugin->endRetrieval();
             }
         }
 
         m_plugin->endRetrieval();

-        m_retrievedItems = true;

     }
 }
 
     }
 }
 

-bool UpdateItemStatesThread::lockPlugin()
-{
-    return m_globalPluginMutex->tryLock(300);
-}
-
-void UpdateItemStatesThread::unlockPlugin()
-{
-    m_globalPluginMutex->unlock();
-}
-
-QList<VersionControlObserver::ItemState> UpdateItemStatesThread::itemStates() const
+QMap<QString, QVector<VersionControlObserver::ItemState> > UpdateItemStatesThread::itemStates() const

 {
 {

-    QMutexLocker locker(&m_itemMutex);

     return m_itemStates;
 }
 
     return m_itemStates;
 }
 

-bool UpdateItemStatesThread::retrievedItems() const
-{
-    QMutexLocker locker(&m_itemMutex);
-    return m_retrievedItems;
-}
-
-#include "updateitemstatesthread.moc"