]> git.sesse.net Git - kdenlive/blobdiff - src/kdenlivedoc.cpp
Reformat initializer lists in all constructors.
[kdenlive] / src / kdenlivedoc.cpp
index 826413acf5b4c511005200d1058d350c0c7dd4fd..a5f40d0b25a503348a2d15b862981330eb008bbd 100644 (file)
 #include <mlt++/Mlt.h>
 
 
-KdenliveDoc::KdenliveDoc(const KUrl &url, const KUrl &projectFolder, QUndoGroup *undoGroup, const QString &profileName, const QPoint tracks, Render *render, MainWindow *parent): QObject(parent), m_render(render), m_url(url), m_projectFolder(projectFolder), m_commandStack(new QUndoStack(undoGroup)), m_modified(false), m_documentLoadingProgress(0), m_documentLoadingStep(0.0), m_startPos(0), m_zoom(7), m_autosave(NULL), m_zoneStart(0), m_zoneEnd(100), m_abortLoading(false) {
+KdenliveDoc::KdenliveDoc(const KUrl &url, const KUrl &projectFolder, QUndoGroup *undoGroup, const QString &profileName, const QPoint tracks, Render *render, MainWindow *parent) :
+        QObject(parent),
+        m_render(render),
+        m_url(url),
+        m_projectFolder(projectFolder),
+        m_commandStack(new QUndoStack(undoGroup)),
+        m_modified(false),
+        m_documentLoadingProgress(0),
+        m_documentLoadingStep(0.0),
+        m_startPos(0),
+        m_zoom(7),
+        m_autosave(NULL),
+        m_zoneStart(0),
+        m_zoneEnd(100),
+        m_abortLoading(false)
+{
     m_clipManager = new ClipManager(this);
     m_autoSaveTimer = new QTimer(this);
     m_autoSaveTimer->setSingleShot(true);
     if (!url.isEmpty()) {
         QString tmpFile;
-        if (KIO::NetAccess::download(url.path(), tmpFile, parent)) {
+        bool success = KIO::NetAccess::download(url.path(), tmpFile, parent);
+        if (success) {
             QFile file(tmpFile);
-            m_document.setContent(&file, false);
+            QString errorMsg;
+            success = m_document.setContent(&file, false, &errorMsg);
             file.close();
+            if (success == false) {
+                // File is corrupted, warn user
+                KMessageBox::error(parent, errorMsg);
+            }
+        } else KMessageBox::error(parent, KIO::NetAccess::lastErrorString());
+
+        if (success) {
             QDomNode infoXmlNode = m_document.elementsByTagName("kdenlivedoc").at(0);
             QDomNode westley = m_document.elementsByTagName("westley").at(0);
             if (!infoXmlNode.isNull()) {
@@ -186,7 +210,6 @@ KdenliveDoc::KdenliveDoc(const KUrl &url, const KUrl &projectFolder, QUndoGroup
             }
             KIO::NetAccess::removeTempFile(tmpFile);
         } else {
-            KMessageBox::error(parent, KIO::NetAccess::lastErrorString());
             parent->slotGotProgressInfo(i18n("File %1 is not a Kdenlive project file."), 100);
             m_url = KUrl();
             m_document = createEmptyDocument(KdenliveSettings::videotracks(), KdenliveSettings::audiotracks());
@@ -212,7 +235,8 @@ KdenliveDoc::KdenliveDoc(const KUrl &url, const KUrl &projectFolder, QUndoGroup
 
 }
 
-KdenliveDoc::~KdenliveDoc() {
+KdenliveDoc::~KdenliveDoc()
+{
     delete m_commandStack;
     delete m_clipManager;
     delete m_autoSaveTimer;
@@ -222,12 +246,14 @@ KdenliveDoc::~KdenliveDoc() {
     }
 }
 
-void KdenliveDoc::setSceneList() {
+void KdenliveDoc::setSceneList()
+{
     m_render->setSceneList(m_document.toString(), m_startPos);
     checkProjectClips();
 }
 
-QDomDocument KdenliveDoc::createEmptyDocument(const int videotracks, const int audiotracks) {
+QDomDocument KdenliveDoc::createEmptyDocument(const int videotracks, const int audiotracks)
+{
     // Creating new document
     QDomDocument doc;
     QDomElement westley = doc.createElement("westley");
@@ -324,26 +350,29 @@ QDomDocument KdenliveDoc::createEmptyDocument(const int videotracks, const int a
 }
 
 
-void KdenliveDoc::syncGuides(QList <Guide *> guides) {
-    QDomDocument doc;
-    QDomElement e;
+void KdenliveDoc::syncGuides(QList <Guide *> guides)
+{
     m_guidesXml.clear();
-    m_guidesXml = doc.createElement("guides");
+    QDomElement guideNode = m_guidesXml.createElement("guides");
+    m_guidesXml.appendChild(guideNode);
+    QDomElement e;
 
     for (int i = 0; i < guides.count(); i++) {
-        e = doc.createElement("guide");
+        e = m_guidesXml.createElement("guide");
         e.setAttribute("time", guides.at(i)->position().ms() / 1000);
         e.setAttribute("comment", guides.at(i)->label());
-        m_guidesXml.appendChild(e);
+        guideNode.appendChild(e);
     }
     emit guidesUpdated();
 }
 
-QDomElement KdenliveDoc::guidesXml() const {
-    return m_guidesXml;
+QDomElement KdenliveDoc::guidesXml() const
+{
+    return m_guidesXml.documentElement();
 }
 
-void KdenliveDoc::slotAutoSave() {
+void KdenliveDoc::slotAutoSave()
+{
     if (m_render && m_autosave) {
         if (!m_autosave->isOpen() && !m_autosave->open(QIODevice::ReadWrite)) {
             // show error: could not open the autosave file
@@ -362,15 +391,18 @@ void KdenliveDoc::slotAutoSave() {
     }
 }
 
-void KdenliveDoc::setZoom(int factor) {
+void KdenliveDoc::setZoom(int factor)
+{
     m_zoom = factor;
 }
 
-int KdenliveDoc::zoom() const {
+int KdenliveDoc::zoom() const
+{
     return m_zoom;
 }
 
-bool KdenliveDoc::convertDocument(double version) {
+bool KdenliveDoc::convertDocument(double version)
+{
     kDebug() << "Opening a document with version " << version;
     const double current_version = 0.82;
 
@@ -454,62 +486,90 @@ bool KdenliveDoc::convertDocument(double version) {
 
     // move playlists outside of tractor and add the tracks instead
     int max = playlists.count();
-    for (int i = 0; i < max; i++) {
-        QDomNode n = playlists.at(i);
-        westley.insertBefore(n, QDomNode());
-        QDomElement pl = n.toElement();
-        QDomElement track = m_document.createElement("track");
-        QString trackType = pl.attribute("hide");
-        if (!trackType.isEmpty())
-            track.setAttribute("hide", trackType);
-        QString playlist_id =  pl.attribute("id");
-        if (playlist_id.isEmpty()) {
-            playlist_id = "black_track";
-            pl.setAttribute("id", playlist_id);
+    if (westley.isNull()) {
+        westley = m_document.createElement("westley");
+        m_document.documentElement().appendChild(westley);
+    }
+    if (tractor.isNull()) {
+        kDebug() << "// NO WESTLEY PLAYLIST, building empty one";
+        QDomElement blank_tractor = m_document.createElement("tractor");
+        westley.appendChild(blank_tractor);
+        QDomElement blank_playlist = m_document.createElement("playlist");
+        blank_playlist.setAttribute("id", "black_track");
+        westley.insertBefore(blank_playlist, QDomNode());
+        QDomElement blank_track = m_document.createElement("track");
+        blank_track.setAttribute("producer", "black_track");
+        blank_tractor.appendChild(blank_track);
+
+        QDomNodeList kdenlivetracks = m_document.elementsByTagName("kdenlivetrack");
+        for (int i = 0; i < kdenlivetracks.count(); i++) {
+            blank_playlist = m_document.createElement("playlist");
+            blank_playlist.setAttribute("id", "playlist" + QString::number(i));
+            westley.insertBefore(blank_playlist, QDomNode());
+            blank_track = m_document.createElement("track");
+            blank_track.setAttribute("producer", "playlist" + QString::number(i));
+            blank_tractor.appendChild(blank_track);
+            if (kdenlivetracks.at(i).toElement().attribute("cliptype") == "Sound") {
+                blank_playlist.setAttribute("hide", "video");
+                blank_track.setAttribute("hide", "video");
+            }
         }
-        track.setAttribute("producer", playlist_id);
-        //tractor.appendChild(track);
+    } else for (int i = 0; i < max; i++) {
+            QDomNode n = playlists.at(i);
+            westley.insertBefore(n, QDomNode());
+            QDomElement pl = n.toElement();
+            QDomElement track = m_document.createElement("track");
+            QString trackType = pl.attribute("hide");
+            if (!trackType.isEmpty())
+                track.setAttribute("hide", trackType);
+            QString playlist_id =  pl.attribute("id");
+            if (playlist_id.isEmpty()) {
+                playlist_id = "black_track";
+                pl.setAttribute("id", playlist_id);
+            }
+            track.setAttribute("producer", playlist_id);
+            //tractor.appendChild(track);
 #define KEEP_TRACK_ORDER 1
 #ifdef KEEP_TRACK_ORDER
-        tractor.insertAfter(track, QDomNode());
+            tractor.insertAfter(track, QDomNode());
 #else
-        // Insert the new track in an order that hopefully matches the 3 video, then 2 audio tracks of Kdenlive 0.7.0
-        // insertion sort - O( tracks*tracks )
-        // Note, this breaks _all_ transitions - but you can move them up and down afterwards.
-        QDomElement tractor_elem = tractor.toElement();
-        if (! tractor_elem.isNull()) {
-            QDomNodeList tracks = tractor_elem.elementsByTagName("track");
-            int size = tracks.size();
-            if (size == 0) {
-                tractor.insertAfter(track, QDomNode());
-            } else {
-                bool inserted = false;
-                for (int i = 0; i < size; ++i) {
-                    QDomElement track_elem = tracks.at(i).toElement();
-                    if (track_elem.isNull()) {
-                        tractor.insertAfter(track, QDomNode());
-                        inserted = true;
-                        break;
-                    } else {
-                        kDebug() << "playlist_id: " << playlist_id << " producer:" << track_elem.attribute("producer");
-                        if (playlist_id < track_elem.attribute("producer")) {
-                            tractor.insertBefore(track, track_elem);
+            // Insert the new track in an order that hopefully matches the 3 video, then 2 audio tracks of Kdenlive 0.7.0
+            // insertion sort - O( tracks*tracks )
+            // Note, this breaks _all_ transitions - but you can move them up and down afterwards.
+            QDomElement tractor_elem = tractor.toElement();
+            if (! tractor_elem.isNull()) {
+                QDomNodeList tracks = tractor_elem.elementsByTagName("track");
+                int size = tracks.size();
+                if (size == 0) {
+                    tractor.insertAfter(track, QDomNode());
+                } else {
+                    bool inserted = false;
+                    for (int i = 0; i < size; ++i) {
+                        QDomElement track_elem = tracks.at(i).toElement();
+                        if (track_elem.isNull()) {
+                            tractor.insertAfter(track, QDomNode());
                             inserted = true;
                             break;
+                        } else {
+                            kDebug() << "playlist_id: " << playlist_id << " producer:" << track_elem.attribute("producer");
+                            if (playlist_id < track_elem.attribute("producer")) {
+                                tractor.insertBefore(track, track_elem);
+                                inserted = true;
+                                break;
+                            }
                         }
                     }
+                    // Reach here, no insertion, insert last
+                    if (!inserted) {
+                        tractor.insertAfter(track, QDomNode());
+                    }
                 }
-                // Reach here, no insertion, insert last
-                if (!inserted) {
-                    tractor.insertAfter(track, QDomNode());
-                }
+            } else {
+                kWarning() << "tractor was not a QDomElement";
+                tractor.insertAfter(track, QDomNode());
             }
-        } else {
-            kWarning() << "tractor was not a QDomElement";
-            tractor.insertAfter(track, QDomNode());
-        }
 #endif
-    }
+        }
     tractor.removeChild(multitrack);
 
     // audio track mixing transitions should not be added to track view, so add required attribute
@@ -523,6 +583,11 @@ bool KdenliveDoc::convertDocument(double version) {
             QDomText value = m_document.createTextNode("237");
             property.appendChild(value);
             tr.appendChild(property);
+            property = m_document.createElement("property");
+            property.setAttribute("name", "mlt_service");
+            value = m_document.createTextNode("mix");
+            property.appendChild(value);
+            tr.appendChild(property);
         } else {
             // convert transition
             QDomNamedNodeMap attrs = tr.attributes();
@@ -634,13 +699,13 @@ bool KdenliveDoc::convertDocument(double version) {
         QDomElement prod = producers.at(0).toElement();
         // add resource also as a property (to allow path correction in setNewResource())
         // TODO: will it work with slowmotion? needs testing
-        if (!prod.attribute("resource").isEmpty()) {
+        /*if (!prod.attribute("resource").isEmpty()) {
             QDomElement prop_resource = m_document.createElement("property");
             prop_resource.setAttribute("name", "resource");
             QDomText resource = m_document.createTextNode(prod.attribute("resource"));
             prop_resource.appendChild(resource);
             prod.appendChild(prop_resource);
-        }
+        }*/
         QDomNode m = prod.firstChild();
         if (!m.isNull()) {
             if (m.toElement().tagName() == "markers") {
@@ -775,37 +840,38 @@ bool KdenliveDoc::convertDocument(double version) {
             QDomElement wproducer = wproducers.at(i).toElement();
             if (wproducer.isNull()) {
                 kWarning() << "Found producer in westley0, that was not a QDomElement";
+                continue;
+            }
+            if (wproducer.attribute("id") == "black") continue;
+            // We have to do slightly different things, depending on the type
+            kDebug() << "Converting producer element with type" << wproducer.attribute("type");
+            if (wproducer.attribute("type").toInt() == TEXT) {
+                kDebug() << "Found TEXT element in producer" << endl;
+                QDomElement kproducer = wproducer.cloneNode(true).toElement();
+                kproducer.setTagName("kdenlive_producer");
+                kdenlivedoc_new.appendChild(kproducer);
+                // TODO: Perhaps needs some more changes here to "frequency", aspect ratio as a float, frame_size, channels, and later, ressource and title name
             } else {
-                // We have to do slightly different things, depending on the type
-                kDebug() << "Converting producer element with type" << wproducer.attribute("type");
-                if (wproducer.attribute("type").toInt() == TEXT) {
-                    kDebug() << "Found TEXT element in producer" << endl;
-                    QDomElement kproducer = wproducer.cloneNode(true).toElement();
-                    kproducer.setTagName("kdenlive_producer");
-                    kdenlivedoc_new.appendChild(kproducer);
-                    // TODO: Perhaps needs some more changes here to "frequency", aspect ratio as a float, frame_size, channels, and later, ressource and title name
-                } else {
-                    QDomElement kproducer = m_document.createElement("kdenlive_producer");
-                    kproducer.setAttribute("id", wproducer.attribute("id"));
-                    if (!wproducer.attribute("description").isEmpty())
-                        kproducer.setAttribute("description", wproducer.attribute("description"));
-                    kproducer.setAttribute("resource", wproducer.attribute("resource"));
-                    kproducer.setAttribute("type", wproducer.attribute("type"));
-                    // Testing fix for 358
-                    if (!wproducer.attribute("aspect_ratio").isEmpty()) {
-                        kproducer.setAttribute("aspect_ratio", wproducer.attribute("aspect_ratio"));
-                    }
-                    if (!wproducer.attribute("source_fps").isEmpty()) {
-                        kproducer.setAttribute("fps", wproducer.attribute("source_fps"));
-                    }
-                    if (!wproducer.attribute("length").isEmpty()) {
-                        kproducer.setAttribute("duration", wproducer.attribute("length"));
-                    }
-                    kdenlivedoc_new.appendChild(kproducer);
+                QDomElement kproducer = m_document.createElement("kdenlive_producer");
+                kproducer.setAttribute("id", wproducer.attribute("id"));
+                if (!wproducer.attribute("description").isEmpty())
+                    kproducer.setAttribute("description", wproducer.attribute("description"));
+                kproducer.setAttribute("resource", wproducer.attribute("resource"));
+                kproducer.setAttribute("type", wproducer.attribute("type"));
+                // Testing fix for 358
+                if (!wproducer.attribute("aspect_ratio").isEmpty()) {
+                    kproducer.setAttribute("aspect_ratio", wproducer.attribute("aspect_ratio"));
+                }
+                if (!wproducer.attribute("source_fps").isEmpty()) {
+                    kproducer.setAttribute("fps", wproducer.attribute("source_fps"));
                 }
-                if (wproducer.attribute("id").toInt() > max_kproducer_id) {
-                    max_kproducer_id = wproducer.attribute("id").toInt();
+                if (!wproducer.attribute("length").isEmpty()) {
+                    kproducer.setAttribute("duration", wproducer.attribute("length"));
                 }
+                kdenlivedoc_new.appendChild(kproducer);
+            }
+            if (wproducer.attribute("id").toInt() > max_kproducer_id) {
+                max_kproducer_id = wproducer.attribute("id").toInt();
             }
         }
     }
@@ -868,7 +934,7 @@ bool KdenliveDoc::convertDocument(double version) {
     kDebug() << "found" << avfiles.count() << "<avfile />s and" << kproducers.count() << "<kdenlive_producer />s";
     for (int i = 0; i < avfiles.count(); ++i) {
         QDomElement avfile = avfiles.at(i).toElement();
-        QDomElement kproducer = QDomElement();
+        QDomElement kproducer;
         if (avfile.isNull())
             kWarning() << "found an <avfile /> that is not a QDomElement";
         else {
@@ -895,39 +961,44 @@ bool KdenliveDoc::convertDocument(double version) {
         }
     }
 
-    //kDebug() << "/////////////////  CONVERTED DOC:";
-    //kDebug() << m_document.toString();
-    /*
-    QFile file( "converted.kdenlive" );
-    if ( file.open( QIODevice::WriteOnly ) ) {
-      QTextStream stream( &file );
-      stream << m_document.toString();
-      file.close();
+    /*kDebug() << "/////////////////  CONVERTED DOC:";
+    kDebug() << m_document.toString();
+    kDebug() << "/////////////////  END CONVERTED DOC:";
+
+    QFile file("converted.kdenlive");
+    if (file.open(QIODevice::WriteOnly)) {
+        QTextStream stream(&file);
+        stream << m_document.toString().toUtf8();
+        file.close();
     } else {
-      kDebug() << "Unable to dump file to converted.kdenlive";
-    }
-    */
+        kDebug() << "Unable to dump file to converted.kdenlive";
+    }*/
+
     //kDebug() << "/////////////////  END CONVERTED DOC:";
 
     return true;
 }
 
-QString KdenliveDoc::colorToString(const QColor& c) {
+QString KdenliveDoc::colorToString(const QColor& c)
+{
     QString ret = "%1,%2,%3,%4";
     ret = ret.arg(c.red()).arg(c.green()).arg(c.blue()).arg(c.alpha());
     return ret;
 }
 
-void KdenliveDoc::setZone(int start, int end) {
+void KdenliveDoc::setZone(int start, int end)
+{
     m_zoneStart = start;
     m_zoneEnd = end;
 }
 
-QPoint KdenliveDoc::zone() const {
+QPoint KdenliveDoc::zone() const
+{
     return QPoint(m_zoneStart, m_zoneEnd);
 }
 
-bool KdenliveDoc::saveSceneList(const QString &path, const QString &scene) {
+bool KdenliveDoc::saveSceneList(const QString &path, const QString &scene)
+{
     QDomDocument sceneList;
     sceneList.setContent(scene, true);
     QDomNode wes = sceneList.elementsByTagName("westley").at(0);
@@ -986,7 +1057,7 @@ bool KdenliveDoc::saveSceneList(const QString &path, const QString &scene) {
     addedXml.appendChild(markers);
 
     // Add guides
-    if (!m_guidesXml.isNull()) addedXml.appendChild(sceneList.importNode(m_guidesXml, true));
+    if (!m_guidesXml.isNull()) addedXml.appendChild(sceneList.importNode(m_guidesXml.documentElement(), true));
 
     // Add clip groups
     addedXml.appendChild(sceneList.importNode(m_clipManager->groupsXml(), true));
@@ -1000,22 +1071,29 @@ bool KdenliveDoc::saveSceneList(const QString &path, const QString &scene) {
         return false;
     }
 
-    QTextStream out(&file);
-    out << sceneList.toString();
+    file.write(sceneList.toString().toUtf8());
+    if (file.error() != QFile::NoError) {
+        KMessageBox::error(kapp->activeWindow(), i18n("Cannot write to file %1", path));
+        file.close();
+        return false;
+    }
     file.close();
     return true;
 }
 
-ClipManager *KdenliveDoc::clipManager() {
+ClipManager *KdenliveDoc::clipManager()
+{
     return m_clipManager;
 }
 
-KUrl KdenliveDoc::projectFolder() const {
+KUrl KdenliveDoc::projectFolder() const
+{
     //if (m_projectFolder.isEmpty()) return KUrl(KStandardDirs::locateLocal("appdata", "/projects/"));
     return m_projectFolder;
 }
 
-void KdenliveDoc::setProjectFolder(KUrl url) {
+void KdenliveDoc::setProjectFolder(KUrl url)
+{
     if (url == m_projectFolder) return;
     setModified(true);
     KStandardDirs::makeDir(url.path());
@@ -1025,7 +1103,8 @@ void KdenliveDoc::setProjectFolder(KUrl url) {
     m_projectFolder = url;
 }
 
-void KdenliveDoc::moveProjectData(KUrl url) {
+void KdenliveDoc::moveProjectData(KUrl url)
+{
     QList <DocClipBase*> list = m_clipManager->documentClipList();
     //TODO: Also move ladspa effects files
     for (int i = 0; i < list.count(); i++) {
@@ -1053,20 +1132,24 @@ void KdenliveDoc::moveProjectData(KUrl url) {
     }
 }
 
-const QString &KdenliveDoc::profilePath() const {
+const QString &KdenliveDoc::profilePath() const
+{
     return m_profile.path;
 }
 
-MltVideoProfile KdenliveDoc::mltProfile() const {
+MltVideoProfile KdenliveDoc::mltProfile() const
+{
     return m_profile;
 }
 
-void KdenliveDoc::setProfilePath(QString path) {
+void KdenliveDoc::setProfilePath(QString path)
+{
     if (path.isEmpty()) path = KdenliveSettings::default_profile();
     if (path.isEmpty()) path = "dv_pal";
     m_profile = ProfilesDialog::getVideoProfile(path);
     KdenliveSettings::setProject_display_ratio((double) m_profile.display_aspect_num / m_profile.display_aspect_den);
     m_fps = (double) m_profile.frame_rate_num / m_profile.frame_rate_den;
+    KdenliveSettings::setProject_fps(m_fps);
     m_width = m_profile.width;
     m_height = m_profile.height;
     kDebug() << "KDEnnlive document, init timecode from path: " << path << ",  " << m_fps;
@@ -1074,20 +1157,24 @@ void KdenliveDoc::setProfilePath(QString path) {
     else m_timecode.setFormat((int) m_fps);
 }
 
-double KdenliveDoc::dar() {
+double KdenliveDoc::dar()
+{
     return (double) m_profile.display_aspect_num / m_profile.display_aspect_den;
 }
 
-void KdenliveDoc::setThumbsProgress(const QString &message, int progress) {
+void KdenliveDoc::setThumbsProgress(const QString &message, int progress)
+{
     emit progressInfo(message, progress);
 }
 
-void KdenliveDoc::loadingProgressed() {
+void KdenliveDoc::loadingProgressed()
+{
     m_documentLoadingProgress += m_documentLoadingStep;
     emit progressInfo(QString(), (int) m_documentLoadingProgress);
 }
 
-QUndoStack *KdenliveDoc::commandStack() {
+QUndoStack *KdenliveDoc::commandStack()
+{
     return m_commandStack;
 }
 
@@ -1105,7 +1192,8 @@ void KdenliveDoc::setRenderer(Render *render) {
     emit progressInfo(QString(), -1);
 }*/
 
-void KdenliveDoc::checkProjectClips() {
+void KdenliveDoc::checkProjectClips()
+{
     if (m_render == NULL) return;
     QList <Mlt::Producer *> prods = m_render->producersList();
     QString id ;
@@ -1145,26 +1233,31 @@ void KdenliveDoc::checkProjectClips() {
     }
 }
 
-void KdenliveDoc::updatePreviewSettings() {
+void KdenliveDoc::updatePreviewSettings()
+{
     m_clipManager->updatePreviewSettings();
     m_render->updatePreviewSettings();
     m_clipManager->resetProducersList(m_render->producersList());
 
 }
 
-Render *KdenliveDoc::renderer() {
+Render *KdenliveDoc::renderer()
+{
     return m_render;
 }
 
-void KdenliveDoc::updateClip(const QString &id) {
+void KdenliveDoc::updateClip(const QString &id)
+{
     emit updateClipDisplay(id);
 }
 
-int KdenliveDoc::getFramePos(QString duration) {
+int KdenliveDoc::getFramePos(QString duration)
+{
     return m_timecode.getFrameCount(duration, m_fps);
 }
 
-QString KdenliveDoc::producerName(const QString &id) {
+QString KdenliveDoc::producerName(const QString &id)
+{
     QString result = "unnamed";
     QDomNodeList prods = producersList();
     int ct = prods.count();
@@ -1179,7 +1272,8 @@ QString KdenliveDoc::producerName(const QString &id) {
     return result;
 }
 
-void KdenliveDoc::setProducerDuration(const QString &id, int duration) {
+void KdenliveDoc::setProducerDuration(const QString &id, int duration)
+{
     QDomNodeList prods = producersList();
     int ct = prods.count();
     for (int i = 0; i <  ct ; i++) {
@@ -1191,7 +1285,8 @@ void KdenliveDoc::setProducerDuration(const QString &id, int duration) {
     }
 }
 
-int KdenliveDoc::getProducerDuration(const QString &id) {
+int KdenliveDoc::getProducerDuration(const QString &id)
+{
     int result = 0;
     QDomNodeList prods = producersList();
     int ct = prods.count();
@@ -1205,54 +1300,56 @@ int KdenliveDoc::getProducerDuration(const QString &id) {
     return result;
 }
 
-
-QDomDocument KdenliveDoc::generateSceneList() {
-    QDomDocument doc;
-    QDomElement westley = doc.createElement("westley");
-    doc.appendChild(westley);
-    QDomElement prod = doc.createElement("producer");
-}
-
-QDomDocument KdenliveDoc::toXml() {
+QDomDocument KdenliveDoc::toXml()
+{
     return m_document;
 }
 
-Timecode KdenliveDoc::timecode() const {
+Timecode KdenliveDoc::timecode() const
+{
     return m_timecode;
 }
 
-QDomNodeList KdenliveDoc::producersList() {
+QDomNodeList KdenliveDoc::producersList()
+{
     return m_document.elementsByTagName("producer");
 }
 
-double KdenliveDoc::projectDuration() const {
+double KdenliveDoc::projectDuration() const
+{
     if (m_render)
         return GenTime(m_render->getLength(), m_fps).ms() / 1000;
     else
         return 0;
 }
 
-double KdenliveDoc::fps() const {
+double KdenliveDoc::fps() const
+{
     return m_fps;
 }
 
-int KdenliveDoc::width() const {
+int KdenliveDoc::width() const
+{
     return m_width;
 }
 
-int KdenliveDoc::height() const {
+int KdenliveDoc::height() const
+{
     return m_height;
 }
 
-KUrl KdenliveDoc::url() const {
+KUrl KdenliveDoc::url() const
+{
     return m_url;
 }
 
-void KdenliveDoc::setUrl(KUrl url) {
+void KdenliveDoc::setUrl(KUrl url)
+{
     m_url = url;
 }
 
-void KdenliveDoc::setModified(bool mod) {
+void KdenliveDoc::setModified(bool mod)
+{
     if (!m_url.isEmpty() && mod && KdenliveSettings::crashrecovery()) {
         m_autoSaveTimer->start(3000);
     }
@@ -1261,18 +1358,21 @@ void KdenliveDoc::setModified(bool mod) {
     emit docModified(m_modified);
 }
 
-bool KdenliveDoc::isModified() const {
+bool KdenliveDoc::isModified() const
+{
     return m_modified;
 }
 
-const QString KdenliveDoc::description() const {
+const QString KdenliveDoc::description() const
+{
     if (m_url.isEmpty())
         return i18n("Untitled") + " / " + m_profile.description;
     else
         return m_url.fileName() + " / " + m_profile.description;
 }
 
-void KdenliveDoc::addClip(QDomElement elem, QString clipId, bool createClipItem) {
+void KdenliveDoc::addClip(QDomElement elem, QString clipId, bool createClipItem)
+{
     const QString producerId = clipId.section('_', 0, 0);
     DocClipBase *clip = m_clipManager->getClipById(producerId);
     if (clip == NULL) {
@@ -1282,8 +1382,7 @@ void KdenliveDoc::addClip(QDomElement elem, QString clipId, bool createClipItem)
         if (elem.attribute("type").toInt() == SLIDESHOW) {
             extension = KUrl(path).fileName();
             path = KUrl(path).directory();
-        }
-        if (elem.attribute("type").toInt() == TEXT && !QFile::exists(path)) {
+        } else if (elem.attribute("type").toInt() == TEXT && QFile::exists(path) == false) {
             kDebug() << "// TITLE: " << elem.attribute("titlename") << " Preview file: " << elem.attribute("resource") << " DOES NOT EXIST";
             QString titlename = elem.attribute("titlename");
             QString titleresource;
@@ -1305,7 +1404,8 @@ void KdenliveDoc::addClip(QDomElement elem, QString clipId, bool createClipItem)
             pix.save(titleresource);
             elem.setAttribute("resource", titleresource);
             delete dia_ui;
-        } else if (!path.isEmpty() && !QFile::exists(path) && elem.attribute("type").toInt() != TEXT) {
+        }
+        if (path.isEmpty() == false && QFile::exists(path) == false && elem.attribute("type").toInt() != TEXT) {
             kDebug() << "// FOUND MISSING CLIP: " << path << ", TYPE: " << elem.attribute("type").toInt();
             const QString size = elem.attribute("file_size");
             const QString hash = elem.attribute("file_hash");
@@ -1362,7 +1462,8 @@ void KdenliveDoc::addClip(QDomElement elem, QString clipId, bool createClipItem)
 }
 
 
-void KdenliveDoc::setNewClipResource(const QString &id, const QString &path) {
+void KdenliveDoc::setNewClipResource(const QString &id, const QString &path)
+{
     QDomNodeList prods = m_document.elementsByTagName("producer");
     int maxprod = prods.count();
     for (int i = 0; i < maxprod; i++) {
@@ -1381,7 +1482,8 @@ void KdenliveDoc::setNewClipResource(const QString &id, const QString &path) {
     }
 }
 
-QString KdenliveDoc::searchFileRecursively(const QDir &dir, const QString &matchSize, const QString &matchHash) const {
+QString KdenliveDoc::searchFileRecursively(const QDir &dir, const QString &matchSize, const QString &matchHash) const
+{
     QString foundFileName;
     QByteArray fileData;
     QByteArray fileHash;
@@ -1417,7 +1519,8 @@ QString KdenliveDoc::searchFileRecursively(const QDir &dir, const QString &match
     return foundFileName;
 }
 
-void KdenliveDoc::addClipInfo(QDomElement elem, QDomElement orig, QString clipId) {
+void KdenliveDoc::addClipInfo(QDomElement elem, QDomElement orig, QString clipId)
+{
     DocClipBase *clip = m_clipManager->getClipById(clipId);
     if (clip == NULL) {
         addClip(elem, clipId, false);
@@ -1451,7 +1554,8 @@ void KdenliveDoc::addClipInfo(QDomElement elem, QDomElement orig, QString clipId
     }
 }
 
-void KdenliveDoc::deleteProjectClip(QList <QString> ids) {
+void KdenliveDoc::deleteProjectClip(QList <QString> ids)
+{
     for (int i = 0; i < ids.size(); ++i) {
         emit deleteTimelineClip(ids.at(i));
         m_clipManager->slotDeleteClip(ids.at(i));
@@ -1459,34 +1563,40 @@ void KdenliveDoc::deleteProjectClip(QList <QString> ids) {
     setModified(true);
 }
 
-void KdenliveDoc::deleteClip(const QString &clipId) {
+void KdenliveDoc::deleteClip(const QString &clipId)
+{
     emit signalDeleteProjectClip(clipId);
     m_clipManager->deleteClip(clipId);
 }
 
-void KdenliveDoc::slotAddClipList(const KUrl::List urls, const QString group, const QString &groupId) {
+void KdenliveDoc::slotAddClipList(const KUrl::List urls, const QString group, const QString &groupId)
+{
     m_clipManager->slotAddClipList(urls, group, groupId);
     emit selectLastAddedClip(QString::number(m_clipManager->lastClipId()));
     setModified(true);
 }
 
 
-void KdenliveDoc::slotAddClipFile(const KUrl url, const QString group, const QString &groupId) {
+void KdenliveDoc::slotAddClipFile(const KUrl url, const QString group, const QString &groupId)
+{
     //kDebug() << "/////////  DOCUM, ADD CLP: " << url;
     m_clipManager->slotAddClipFile(url, group, groupId);
     emit selectLastAddedClip(QString::number(m_clipManager->lastClipId()));
     setModified(true);
 }
 
-const QString KdenliveDoc::getFreeClipId() {
+const QString KdenliveDoc::getFreeClipId()
+{
     return QString::number(m_clipManager->getFreeClipId());
 }
 
-DocClipBase *KdenliveDoc::getBaseClip(const QString &clipId) {
+DocClipBase *KdenliveDoc::getBaseClip(const QString &clipId)
+{
     return m_clipManager->getClipById(clipId);
 }
 
-void KdenliveDoc::slotCreateTextClip(QString /*group*/, const QString &/*groupId*/) {
+void KdenliveDoc::slotCreateTextClip(QString /*group*/, const QString &/*groupId*/)
+{
     QString titlesFolder = projectFolder().path() + "/titles/";
     KStandardDirs::makeDir(titlesFolder);
     TitleWidget *dia_ui = new TitleWidget(KUrl(), titlesFolder, m_render, kapp->activeWindow());
@@ -1501,51 +1611,62 @@ void KdenliveDoc::slotCreateTextClip(QString /*group*/, const QString &/*groupId
     delete dia_ui;
 }
 
-int KdenliveDoc::tracksCount() const {
+int KdenliveDoc::tracksCount() const
+{
     return m_tracksList.count();
 }
 
-TrackInfo KdenliveDoc::trackInfoAt(int ix) const {
+TrackInfo KdenliveDoc::trackInfoAt(int ix) const
+{
     return m_tracksList.at(ix);
 }
 
-void KdenliveDoc::switchTrackAudio(int ix, bool hide) {
+void KdenliveDoc::switchTrackAudio(int ix, bool hide)
+{
     m_tracksList[ix].isMute = hide; // !m_tracksList.at(ix).isMute;
 }
 
-void KdenliveDoc::switchTrackLock(int ix, bool lock) {
+void KdenliveDoc::switchTrackLock(int ix, bool lock)
+{
     m_tracksList[ix].isLocked = lock;
 }
 
-bool KdenliveDoc::isTrackLocked(int ix) const {
-    return m_tracksList[ix].isLocked;
+bool KdenliveDoc::isTrackLocked(int ix) const
+{
+    return m_tracksList.at(ix).isLocked;
 }
 
-void KdenliveDoc::switchTrackVideo(int ix, bool hide) {
+void KdenliveDoc::switchTrackVideo(int ix, bool hide)
+{
     m_tracksList[ix].isBlind = hide; // !m_tracksList.at(ix).isBlind;
 }
 
-void KdenliveDoc::insertTrack(int ix, TrackInfo type) {
+void KdenliveDoc::insertTrack(int ix, TrackInfo type)
+{
     if (ix == -1) m_tracksList << type;
     else m_tracksList.insert(ix, type);
 }
 
-void KdenliveDoc::deleteTrack(int ix) {
+void KdenliveDoc::deleteTrack(int ix)
+{
     m_tracksList.removeAt(ix);
 }
 
-void KdenliveDoc::setTrackType(int ix, TrackInfo type) {
+void KdenliveDoc::setTrackType(int ix, TrackInfo type)
+{
     m_tracksList[ix].type = type.type;
     m_tracksList[ix].isMute = type.isMute;
     m_tracksList[ix].isBlind = type.isBlind;
     m_tracksList[ix].isLocked = type.isLocked;
 }
 
-const QList <TrackInfo> KdenliveDoc::tracksList() const {
+const QList <TrackInfo> KdenliveDoc::tracksList() const
+{
     return m_tracksList;
 }
 
-QPoint KdenliveDoc::getTracksCount() const {
+QPoint KdenliveDoc::getTracksCount() const
+{
     int audio = 0;
     int video = 0;
     foreach(const TrackInfo &info, m_tracksList) {
@@ -1555,11 +1676,13 @@ QPoint KdenliveDoc::getTracksCount() const {
     return QPoint(video, audio);
 }
 
-void KdenliveDoc::cachePixmap(const QString &fileId, const QPixmap &pix) const {
+void KdenliveDoc::cachePixmap(const QString &fileId, const QPixmap &pix) const
+{
     pix.save(m_projectFolder.path() + "/thumbs/" + fileId + ".png");
 }
 
-QString KdenliveDoc::getLadspaFile() const {
+QString KdenliveDoc::getLadspaFile() const
+{
     int ct = 0;
     QString counter = QString::number(ct).rightJustified(5, '0', false);
     while (QFile::exists(m_projectFolder.path() + "/ladspa/" + counter + ".ladspa")) {