Use KLocalizedString (for i18n only, in kf5 it will necessary => use a script for...

[kdenlive] / src / commands / moveeffectcommand.cpp

diff --git a/src/commands/moveeffectcommand.cpp b/src/commands/moveeffectcommand.cpp
index ba394793b0ac2aefb9d047240fc0758d3fb6dcd8..4d34cd0eadf5a5d2dbfd9511f4e2a1653ec0f617 100644 (file)
--- a/src/commands/moveeffectcommand.cpp
+++ b/src/commands/moveeffectcommand.cpp
@@ -21,17 +21,18 @@
 #include "moveeffectcommand.h"
 #include "customtrackview.h"
 
-#include <KLocale>
+#include <KLocalizedString>
+#include <KDebug>
 
 MoveEffectCommand::MoveEffectCommand(CustomTrackView *view, const int track, const GenTime &pos, const QList<int> &oldPos, int newPos, QUndoCommand * parent) :
-        QUndoCommand(parent),
-        m_view(view),
-        m_track(track),
-        m_oldindex(oldPos),
-        m_pos(pos)
+    QUndoCommand(parent),
+    m_view(view),
+    m_track(track),
+    m_oldindex(oldPos),
+    m_pos(pos)
 {
-    for (int i = 0; i < m_oldindex.count(); i++) {
-       m_newindex << newPos + i;
+    for (int i = 0; i < m_oldindex.count(); ++i) {
+        m_newindex << newPos + i;
     }
     /*    QString effectName;
         QDomElement namenode = effect.firstChildElement("name");
@@ -51,12 +52,14 @@ int MoveEffectCommand::id() const
 bool MoveEffectCommand::mergeWith(const QUndoCommand * other)
 {
     return false;
+    /* dead code (flaged by coverity), was removed to avoid crash when dropping on group
     if (other->id() != id()) return false;
     if (m_track != static_cast<const MoveEffectCommand*>(other)->m_track) return false;
     if (m_pos != static_cast<const MoveEffectCommand*>(other)->m_pos) return false;
     m_oldindex = static_cast<const MoveEffectCommand*>(other)->m_oldindex;
     m_newindex = static_cast<const MoveEffectCommand*>(other)->m_newindex;
     return true;
+    */
 }
 
 // virtual