Fix some avformat producer concurrency crashes

[kdenlive] / src / addeffectcommand.cpp

diff --git a/src/addeffectcommand.cpp b/src/addeffectcommand.cpp
index cceed84386343ce6572e9a948acd7242ee8e9698..5e09957c7890a8f253eabfeee1b16773dcb6bc48 100644 (file)
--- a/src/addeffectcommand.cpp
+++ b/src/addeffectcommand.cpp
@@ -17,16 +17,23 @@
  *   51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA          *
  ***************************************************************************/
 
-#include <KLocale>
 
 #include "addeffectcommand.h"
 #include "customtrackview.h"
 
-AddEffectCommand::AddEffectCommand(CustomTrackView *view, const int track, GenTime pos, QDomElement effect, bool doIt)
-        : m_view(view), m_track(track), m_pos(pos), m_effect(effect), m_doIt(doIt) {
+#include <KLocale>
+
+AddEffectCommand::AddEffectCommand(CustomTrackView *view, const int track, GenTime pos, QDomElement effect, bool doIt, QUndoCommand * parent) :
+        QUndoCommand(parent),
+        m_view(view),
+        m_track(track),
+        m_effect(effect.cloneNode().toElement()),
+        m_pos(pos),
+        m_doIt(doIt)
+{
     QString effectName;
-    QDomNode namenode = effect.elementsByTagName("name").item(0);
-    if (!namenode.isNull()) effectName = i18n(namenode.toElement().text().toUtf8().data());
+    QDomElement namenode = m_effect.firstChildElement("name");
+    if (!namenode.isNull()) effectName = i18n(namenode.text().toUtf8().data());
     else effectName = i18n("effect");
     if (doIt) setText(i18n("Add %1", effectName));
     else setText(i18n("Delete %1", effectName));
@@ -34,16 +41,18 @@ AddEffectCommand::AddEffectCommand(CustomTrackView *view, const int track, GenTi
 
 
 // virtual
-void AddEffectCommand::undo() {
+void AddEffectCommand::undo()
+{
     kDebug() << "----  undoing action";
     if (m_doIt) m_view->deleteEffect(m_track, m_pos, m_effect);
     else m_view->addEffect(m_track, m_pos, m_effect);
 }
 // virtual
-void AddEffectCommand::redo() {
+void AddEffectCommand::redo()
+{
     kDebug() << "----  redoing action";
     if (m_doIt) m_view->addEffect(m_track, m_pos, m_effect);
     else m_view->deleteEffect(m_track, m_pos, m_effect);
 }
 
-#include "addeffectcommand.moc"
+