meltjob.cpp: initialize m_showFrameEvent [Coverity 3/14] by Mikko Rapeli

[kdenlive] / src / projecttree / meltjob.cpp

diff --git a/src/projecttree/meltjob.cpp b/src/projecttree/meltjob.cpp
index 7e239e206f8ce20a65a753736a61442d96de9262..19d29e144ad24246dff213d91de5444f4ec8e4b9 100644 (file)
--- a/src/projecttree/meltjob.cpp
+++ b/src/projecttree/meltjob.cpp
@@ -38,11 +38,14 @@ MeltJob::MeltJob(CLIPTYPE cType, const QString &id, QStringList parameters) : Ab
     m_producer(NULL),
     m_profile(NULL),
     m_consumer(NULL),
+    m_showFrameEvent(NULL),
     m_length(0)
 {
     jobStatus = JOBWAITING;
     m_params = parameters;
     description = i18n("Process clip");
+    QString consum = m_params.at(5);
+    if (consum.contains(':')) m_dest = consum.section(':', 1);
 }
 
 void MeltJob::setProducer(Mlt::Producer *producer)
@@ -57,56 +60,105 @@ void MeltJob::startJob()
         setStatus(JOBCRASHED);
         return;
     }
+    int in = m_params.takeFirst().toInt();
+    int out = m_params.takeFirst().toInt();
+    QString producerParams =m_params.takeFirst(); 
     QString filter = m_params.takeFirst();
     QString filterParams = m_params.takeFirst();
     QString consumer = m_params.takeFirst();
+    kDebug()<<"consumer: "<<consumer;
+    if (consumer.contains(':')) m_dest = consumer.section(':', 1);
     QString consumerParams = m_params.takeFirst();
-    QString properties = m_params.takeFirst();
+    
+    // optional params
+    QString properties;
+    if (!m_params.isEmpty()) properties = m_params.takeFirst();
+    int startPos = -1;
+    if (!m_params.isEmpty()) startPos = m_params.takeFirst().toInt();
+    int track = -1;
+    if (!m_params.isEmpty()) track = m_params.takeFirst().toInt();
+    QString finalFilter;
+    if (!m_params.isEmpty()) finalFilter = m_params.takeFirst();
+    else finalFilter = filter;
+
+    if (out != -1 && out <= in) {
+        m_errorMessage.append(i18n("Clip zone undefined (%1 - %2).", in, out));
+        setStatus(JOBCRASHED);
+        return;
+    }
+    
     m_profile = m_producer->profile();
-    m_consumer = new Mlt::Consumer(*m_profile, consumer.toUtf8().constData());
+
+    Mlt::Producer *prod;
+    if (out == -1) {
+        QString url = QString::fromUtf8(m_producer->get("resource"));
+        prod = new Mlt::Producer(*m_profile,  url.toUtf8().constData());
+    }
+    else 
+        prod = m_producer->cut(in, out);
+    QStringList list = producerParams.split(' ', QString::SkipEmptyParts);
+    foreach(const QString &data, list) {
+        if (data.contains('=')) {
+            prod->set(data.section('=', 0, 0).toUtf8().constData(), data.section('=', 1, 1).toUtf8().constData());
+        }
+    }
+
+    if (consumer.contains(":")) {
+        m_consumer = new Mlt::Consumer(*m_profile, consumer.section(':', 0, 0).toUtf8().constData(), consumer.section(':', 1).toUtf8().constData());
+    }
+    else {
+        m_consumer = new Mlt::Consumer(*m_profile, consumer.toUtf8().constData());
+    }
     if (!m_consumer || !m_consumer->is_valid()) {
         m_errorMessage.append(i18n("Cannot create consumer %1.", consumer));
         setStatus(JOBCRASHED);
         return;
     }
 
-    m_consumer->set("terminate_on_pause", 1 );
-    m_consumer->set("eof", "pause" );
-    m_consumer->set("real_time", -1 );
+    //m_consumer->set("terminate_on_pause", 1 );
+    //m_consumer->set("eof", "pause" );
+    m_consumer->set("real_time", -KdenliveSettings::mltthreads() );
 
-    QStringList list = consumerParams.split(' ', QString::SkipEmptyParts);
-    foreach(QString data, list) {
+    list = consumerParams.split(' ', QString::SkipEmptyParts);
+    foreach(const QString &data, list) {
         if (data.contains('=')) {
+            kDebug()<<"// filter con: "<<data;
             m_consumer->set(data.section('=', 0, 0).toUtf8().constData(), data.section('=', 1, 1).toUtf8().constData());
         }
     }
     
     Mlt::Filter mltFilter(*m_profile, filter.toUtf8().data());
     list = filterParams.split(' ', QString::SkipEmptyParts);
-    foreach(QString data, list) {
+    foreach(const QString &data, list) {
         if (data.contains('=')) {
+            kDebug()<<"// filter p: "<<data;
             mltFilter.set(data.section('=', 0, 0).toUtf8().constData(), data.section('=', 1, 1).toUtf8().constData());
         }
     }
-    m_producer->attach(mltFilter);
-    m_length = m_producer->get_length();
-    m_consumer->connect(*m_producer);
-    m_producer->set_speed(0);
-    m_producer->seek(0);
+    Mlt::Playlist playlist;
+    playlist.append(*prod);
+    m_length = prod->get_length();
+    m_consumer->connect(playlist);
+    prod->set_speed(0);
+    prod->seek(0);
+    prod->attach(mltFilter);
     m_showFrameEvent = m_consumer->listen("consumer-frame-show", this, (mlt_listener) consumer_frame_render);
     m_consumer->start();
-    m_producer->set_speed(1);
+    prod->set_speed(1);
     while (jobStatus != JOBABORTED && !m_consumer->is_stopped()) {
         
     }
     m_consumer->stop();
     QStringList wanted = properties.split(',', QString::SkipEmptyParts);
-    foreach(const QString key, wanted) {
+    stringMap jobResults;
+    foreach(const QString &key, wanted) {
         QString value = mltFilter.get(key.toUtf8().constData());
-        kDebug()<<"RESULT: "<<key<<" = "<< value;
+        jobResults.insert(key, value);
     }
+    if (!jobResults.isEmpty()) emit gotFilterJobResults(m_clipId, startPos, track, finalFilter, jobResults);
     setStatus(JOBDONE);
     delete m_consumer;
+    delete prod;
     return;
 }
 
@@ -117,7 +169,7 @@ MeltJob::~MeltJob()
 
 const QString MeltJob::destination() const
 {
-    return QString();
+    return m_dest;
 }
 
 stringMap MeltJob::cancelProperties()
@@ -131,10 +183,10 @@ const QString MeltJob::statusMessage()
     QString statusInfo;
     switch (jobStatus) {
         case JOBWORKING:
-            statusInfo = i18n("Processing clip");
+            statusInfo = description;
             break;
         case JOBWAITING:
-            statusInfo = i18n("Waiting - process clip");
+            statusInfo = i18n("Waiting to process clip");
             break;
         default:
             break;
@@ -147,4 +199,5 @@ void MeltJob::emitFrameNumber()
     if (m_consumer && m_length > 0) {
         emit jobProgress(m_clipId, (int) (100 * m_consumer->position() / m_length), jobType);
     }
-}
\ No newline at end of file
+}
+