[Kdenlive-svn] SF.net SVN: kdenlive:[5970] trunk/kdenlive/src/docclipbase.cpp

[<j-...@us...>]

Revision: 5970
          http://kdenlive.svn.sourceforge.net/kdenlive/?rev=5970&view=rev
Author:   j-b-m
Date:     2011-10-18 16:37:47 +0000 (Tue, 18 Oct 2011)
Log Message:
-----------
Fix another possible memleak

Modified Paths:
--------------
    trunk/kdenlive/src/docclipbase.cpp

Modified: trunk/kdenlive/src/docclipbase.cpp
===================================================================
--- trunk/kdenlive/src/docclipbase.cpp	2011-10-18 15:29:59 UTC (rev 5969)
+++ trunk/kdenlive/src/docclipbase.cpp	2011-10-18 16:37:47 UTC (rev 5970)
@@ -443,6 +443,20 @@
 
 void DocClipBase::cleanupProducers()
 {
+    /*
+    int ct = 0;
+    kDebug()<<"----------------------------------------------------------------------------------";
+    for (int i = 0; i < m_toDeleteProducers.count(); i++) {
+        if (m_toDeleteProducers.at(i) != NULL) {
+            Mlt::Properties props(m_toDeleteProducers.at(i)->get_properties());
+            if (props.ref_count() > 2) {
+                kDebug()<<"PRODUCER: "<<i<<", COUNTS: "<<props.ref_count();
+                //exit(1);
+            }
+            ct++;
+        }
+    }*/
+
     qDeleteAll(m_toDeleteProducers);
     m_toDeleteProducers.clear();
     m_replaceMutex.unlock();
@@ -496,12 +510,14 @@
                 m_audioTrackProducers[pos] = producer;
                 updated = true;
             }
+            else delete producer;
             return;
         } else if (id.endsWith("video")) {
             if (m_videoOnlyProducer == NULL) {
                 m_videoOnlyProducer = producer;
                 updated = true;
             }
+            else delete producer;
             return;
         }
         int pos = id.toInt();
@@ -514,6 +530,7 @@
             m_baseTrackProducers[pos] = producer;
             updated = true;
         }
+        else delete producer;
     } else {
         if (m_baseTrackProducers.isEmpty()) {
             m_baseTrackProducers.append(producer);
@@ -523,6 +540,7 @@
             m_baseTrackProducers[0] = producer;
             updated = true;
         }
+        else delete producer;
     }
     if (updated && readPropertiesFromProducer && (m_clipType != COLOR && m_clipType != IMAGE && m_clipType != TEXT))
         setDuration(GenTime(producer->get_length(), KdenliveSettings::project_fps()));

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.