]> git.sesse.net Git - kdenlive/blobdiff - src/kthumb.cpp
[PATCH 3/6] Don't bother null checking before calling delete.
[kdenlive] / src / kthumb.cpp
index 88aa4492857a10158eeae35e81085461059fce7d..0e8435fdc599643356be9664e631ea7d1cb2873c 100644 (file)
  *                                                                         *
  ***************************************************************************/
 
-#include <qxml.h>
-#include <QImage>
-#include <QApplication>
+#include "kthumb.h"
+#include "clipmanager.h"
+#include "renderer.h"
+#include "kdenlivesettings.h"
+
+#include <mlt++/Mlt.h>
 
 #include <kio/netaccess.h>
 #include <kdebug.h>
 #include <klocale.h>
-#include <kdenlivesettings.h>
 #include <kfileitem.h>
 #include <kmessagebox.h>
 #include <KStandardDirs>
 
-#include <mlt++/Mlt.h>
-
-#include "clipmanager.h"
-#include "renderer.h"
-#include "kthumb.h"
-#include "kdenlivesettings.h"
-
+#include <qxml.h>
+#include <QImage>
+#include <QApplication>
 
-void MyThread::init(QObject *parent, KUrl url, QString target, double frame, double frameLength, int frequency, int channels, int arrayWidth) {
+void MyThread::init(KUrl url, QString target, double frame, double frameLength, int frequency, int channels, int arrayWidth)
+{
     stop_me = false;
-    m_parent = parent;
     m_isWorking = false;
     f.setFileName(target);
     m_url = url;
@@ -53,11 +51,13 @@ void MyThread::init(QObject *parent, KUrl url, QString target, double frame, dou
     m_arrayWidth = arrayWidth;
 }
 
-bool MyThread::isWorking() {
+bool MyThread::isWorking()
+{
     return m_isWorking;
 }
 
-void MyThread::run() {
+void MyThread::run()
+{
     if (!f.open(QIODevice::WriteOnly)) {
         kDebug() << "++++++++  ERROR WRITING TO FILE: " << f.fileName() << endl;
         kDebug() << "++++++++  DISABLING AUDIO THUMBS" << endl;
@@ -75,18 +75,14 @@ void MyThread::run() {
         m_producer.attach(m_convert);
     }
 
-    //QApplication::postEvent(m_parent, new ProgressEvent(-1, (QEvent::Type)10005));
-
     int last_val = 0;
     int val = 0;
     kDebug() << "for " << m_frame << " " << m_frameLength << " " << m_producer.is_valid();
     for (int z = (int) m_frame;z < (int)(m_frame + m_frameLength) && m_producer.is_valid();z++) {
         if (stop_me) break;
         val = (int)((z - m_frame) / (m_frame + m_frameLength) * 100.0);
-        if (last_val != val & val > 1) {
+        if (last_val != val && val > 1) {
             emit audioThumbProgress(val);
-            //QApplication::postEvent(m_parent, new ProgressEvent(val, (QEvent::Type)10005));
-
             last_val = val;
         }
         m_producer.seek(z);
@@ -96,12 +92,12 @@ void MyThread::run() {
             int m_samples = mlt_sample_calculator(m_framesPerSecond, m_frequency, mlt_frame_get_position(mlt_frame->get_frame()));
             mlt_audio_format m_audioFormat = mlt_audio_pcm;
 
-            int16_t* m_pcm = mlt_frame->get_audio(m_audioFormat, m_frequency, m_channels, m_samples);
+            qint16* m_pcm = mlt_frame->get_audio(m_audioFormat, m_frequency, m_channels, m_samples);
 
             for (int c = 0;c < m_channels;c++) {
                 QByteArray m_array;
                 m_array.resize(m_arrayWidth);
-                for (uint i = 0; i < m_array.size(); i++) {
+                for (int i = 0; i < m_array.size(); i++) {
                     m_array[i] = ((*(m_pcm + c + i * m_samples / m_array.size())) >> 9) + 127 / 2 ;
                 }
                 f.write(m_array);
@@ -110,56 +106,85 @@ void MyThread::run() {
         } else {
             f.write(QByteArray(m_arrayWidth, '\x00'));
         }
-        if (mlt_frame)
-            delete mlt_frame;
+        delete mlt_frame;
     }
-    kDebug() << "done";
+    //kDebug() << "done";
     f.close();
     m_isWorking = false;
     if (stop_me) {
         f.remove();
-    }
-    emit audioThumbOver();
-    //QApplication::postEvent(m_parent, new ProgressEvent(-1, (QEvent::Type)10005));
-
+    } else emit audioThumbOver();
 }
 
-KThumb::KThumb(ClipManager *clipManager, KUrl url, const QString &id, const QString &hash, QObject * parent, const char *name): QObject(parent), m_clipManager(clipManager), m_url(url), m_id(id), m_producer(NULL), m_dar(1), m_mainFrame(-1) {
+KThumb::KThumb(ClipManager *clipManager, KUrl url, const QString &id, const QString &hash, QObject * parent, const char */*name*/) :
+        QObject(parent),
+        audioThumbProducer(),
+        m_url(url),
+        m_thumbFile(),
+        m_dar(1),
+        m_producer(NULL),
+        m_clipManager(clipManager),
+        m_id(id),
+        m_mainFrame(-1)
+{
     m_thumbFile = clipManager->projectFolder() + "/thumbs/" + hash + ".thumb";
     connect(&audioThumbProducer, SIGNAL(audioThumbProgress(const int)), this, SLOT(slotAudioThumbProgress(const int)));
     connect(&audioThumbProducer, SIGNAL(audioThumbOver()), this, SLOT(slotAudioThumbOver()));
 
 }
 
-KThumb::~KThumb() {
-    if (audioThumbProducer.isRunning()) audioThumbProducer.exit();
+KThumb::~KThumb()
+{
+    if (audioThumbProducer.isRunning()) {
+        audioThumbProducer.stop_me = true;
+        audioThumbProducer.wait();
+        slotAudioThumbOver();
+    }
 }
 
-void KThumb::setProducer(Mlt::Producer *producer) {
+void KThumb::setProducer(Mlt::Producer *producer)
+{
     m_producer = producer;
+    // FIXME: the profile() call leaks an object, but trying to free
+    // it leads to a double-free in Profile::~Profile()
     m_dar = producer->profile()->dar();
 }
 
-bool KThumb::hasProducer() const {
+void KThumb::clearProducer()
+{
+    m_producer = NULL;
+}
+
+bool KThumb::hasProducer() const
+{
     return m_producer != NULL;
 }
 
-void KThumb::updateClipUrl(KUrl url) {
+void KThumb::updateThumbUrl(const QString &hash)
+{
+    m_thumbFile = m_clipManager->projectFolder() + "/thumbs/" + hash + ".thumb";
+}
+
+void KThumb::updateClipUrl(KUrl url, const QString &hash)
+{
     m_url = url;
     if (m_producer) {
         char *tmp = Render::decodedString(url.path());
         m_producer->set("resource", tmp);
         delete[] tmp;
     }
+    m_thumbFile = m_clipManager->projectFolder() + "/thumbs/" + hash + ".thumb";
 }
 
 //static
-QPixmap KThumb::getImage(KUrl url, int width, int height) {
+QPixmap KThumb::getImage(KUrl url, int width, int height)
+{
     if (url.isEmpty()) return QPixmap();
     return getImage(url, 0, width, height);
 }
 
-void KThumb::extractImage(int frame, int frame2) {
+void KThumb::extractImage(int frame, int frame2)
+{
     if (m_url.isEmpty() || !KdenliveSettings::videothumbnails() || m_producer == NULL) return;
 
     const int twidth = (int)(KdenliveSettings::trackheight() * m_dar);
@@ -238,12 +263,14 @@ void KThumb::extractImage(int frame, int frame2) {
     }
 }
 
-QPixmap KThumb::extractImage(int frame, int width, int height) {
+QPixmap KThumb::extractImage(int frame, int width, int height)
+{
     return getFrame(m_producer, frame, width, height);
 }
 
 //static
-QPixmap KThumb::getImage(KUrl url, int frame, int width, int height) {
+QPixmap KThumb::getImage(KUrl url, int frame, int width, int height)
+{
     Mlt::Profile profile((char*) KdenliveSettings::current_profile().data());
     QPixmap pix(width, height);
     if (url.isEmpty()) return pix;
@@ -287,7 +314,8 @@ QPixmap KThumb::getImage(QDomElement xml, int frame, int width, int height) {
 }*/
 
 //static
-QPixmap KThumb::getFrame(Mlt::Producer *producer, int framepos, int width, int height) {
+QPixmap KThumb::getFrame(Mlt::Producer *producer, int framepos, int width, int height)
+{
     if (producer == NULL) {
         QPixmap p(width, height);
         p.fill(Qt::red);
@@ -399,18 +427,24 @@ void KThumb::getThumbs(KUrl url, int startframe, int endframe, int width, int he
     emit thumbReady(endframe, image);
 }
 */
-void KThumb::stopAudioThumbs() {
-    if (audioThumbProducer.isRunning()) audioThumbProducer.stop_me = true;
+void KThumb::stopAudioThumbs()
+{
+    if (audioThumbProducer.isRunning()) {
+        audioThumbProducer.stop_me = true;
+        slotAudioThumbOver();
+    }
 }
 
-void KThumb::removeAudioThumb() {
+void KThumb::removeAudioThumb()
+{
     if (m_thumbFile.isEmpty()) return;
     stopAudioThumbs();
     QFile f(m_thumbFile);
     f.remove();
 }
 
-void KThumb::getAudioThumbs(int channel, double frame, double frameLength, int arrayWidth) {
+void KThumb::getAudioThumbs(int channel, double frame, double frameLength, int arrayWidth)
+{
     if (channel == 0) {
         slotAudioThumbOver();
         return;
@@ -447,22 +481,25 @@ void KThumb::getAudioThumbs(int channel, double frame, double frameLength, int a
         slotAudioThumbOver();
     } else {
         if (audioThumbProducer.isRunning()) return;
-        audioThumbProducer.init(this, m_url, m_thumbFile, frame, frameLength, m_frequency, m_channels, arrayWidth);
+        audioThumbProducer.init(m_url, m_thumbFile, frame, frameLength, m_frequency, m_channels, arrayWidth);
         audioThumbProducer.start(QThread::LowestPriority);
-        kDebug() << "STARTING GENERATE THMB FOR: " << m_url << " ................................";
+        kDebug() << "STARTING GENERATE THMB FOR: " << m_url << " ................................";
     }
 }
 
-void KThumb::slotAudioThumbProgress(const int progress) {
+void KThumb::slotAudioThumbProgress(const int progress)
+{
     m_clipManager->setThumbsProgress(i18n("Creating thumbnail for %1", m_url.fileName()), progress);
 }
 
-void KThumb::slotAudioThumbOver() {
+void KThumb::slotAudioThumbOver()
+{
     m_clipManager->setThumbsProgress(i18n("Creating thumbnail for %1", m_url.fileName()), -1);
     m_clipManager->endAudioThumbsGeneration(m_id);
 }
 
-void KThumb::askForAudioThumbs(const QString &id) {
+void KThumb::askForAudioThumbs(const QString &id)
+{
     m_clipManager->askForAudioThumb(id);
 }