Fix disabling proxy on a clip when it crashed

[kdenlive] / src / clipmanager.cpp

diff --git a/src/clipmanager.cpp b/src/clipmanager.cpp
index f4d22a5cfc55a587e484abacb8e29b9021b40944..cdd3eb0df2ad0d209a3b19a5841f4abf20f7bff3 100644 (file)
--- a/src/clipmanager.cpp
+++ b/src/clipmanager.cpp
@@ -134,22 +134,22 @@ void ClipManager::stopThumbs(const QString &id)
     m_requestedThumbs.remove(id);
     m_thumbsMutex.unlock();
     m_abortThumb = false;
-    if (!m_thumbsThread.isRunning()) {
+    if (!m_thumbsThread.isRunning() && !m_requestedThumbs.isEmpty()) {
         m_thumbsThread = QtConcurrent::run(this, &ClipManager::slotGetThumbs);
     }
 }
 
 void ClipManager::slotGetThumbs()
 {
-    QMap<QString, int>::iterator i = m_requestedThumbs.begin();
+    QMap<QString, int>::const_iterator i;
     int max;
     int done = 0;
-    while (i != m_requestedThumbs.end() && !m_abortThumb) {
-        QString producerId = i.key();
+    while (!m_requestedThumbs.isEmpty() && !m_abortThumb) {
         m_thumbsMutex.lock();
+        i = m_requestedThumbs.constBegin();
+        QString producerId = i.key();
         QList<int> values = m_requestedThumbs.values(producerId);
         m_requestedThumbs.remove(producerId);
-        i = m_requestedThumbs.begin();
         m_thumbsMutex.unlock();
         qSort(values);
         DocClipBase *clip = getClipById(producerId);
@@ -280,7 +280,7 @@ void ClipManager::slotGetAudioThumbs()
         for (int z = (int) frame; z < (int)(frame + lengthInFrames) && producer.is_valid() &&  !m_abortAudioThumb; z++) {
             val = (int)((z - frame) / (frame + lengthInFrames) * 100.0);
             if (last_val != val && val > 1) {
-                setThumbsProgress(i18n("Creating thumbnail for %1", url.fileName()), val);
+                setThumbsProgress(i18n("Creating audio thumbnail for %1", url.fileName()), val);
                 last_val = val;
             }
             producer.seek(z);
@@ -303,8 +303,7 @@ void ClipManager::slotGetAudioThumbs()
             delete mlt_frame;
         }
         f.close();
-        //TODO: post 8.2.1, change text to AUDIO thumbnails
-        setThumbsProgress(i18n("Creating thumbnail for %1", url.fileName()), -1);
+        setThumbsProgress(i18n("Creating audio thumbnail for %1", url.fileName()), -1);
         if (m_abortAudioThumb) {
             f.remove();
         } else {