Better to use ++i than i++ (minor optimization)

[kdenlive] / src / effectstack / collapsiblegroup.cpp

diff --git a/src/effectstack/collapsiblegroup.cpp b/src/effectstack/collapsiblegroup.cpp
index d0d5d3de570a064aa671487de72dfc39175432a9..0e0c873ec2e3b90f6aafcf668213f0408be902a7 100644 (file)
--- a/src/effectstack/collapsiblegroup.cpp
+++ b/src/effectstack/collapsiblegroup.cpp
@@ -135,7 +135,7 @@ void CollapsibleGroup::slotEnable(bool disable, bool emitInfo)
     enabledButton->setChecked(disable);
     enabledButton->setIcon(disable ? KIcon("novisible") : KIcon("visible"));
     enabledButton->blockSignals(false);
-    for (int i = 0; i < m_subWidgets.count(); i++)
+    for (int i = 0; i < m_subWidgets.count(); ++i)
        m_subWidgets.at(i)->slotDisable(disable, emitInfo);
 }
 
@@ -152,7 +152,7 @@ void CollapsibleGroup::slotDeleteGroup()
 void CollapsibleGroup::slotEffectUp()
 {
     QList <int> indexes;
-    for (int i = 0; i < m_subWidgets.count(); i++)
+    for (int i = 0; i < m_subWidgets.count(); ++i)
         indexes << m_subWidgets.at(i)->effectIndex();
     emit changeEffectPosition(indexes, true);
 }
@@ -160,7 +160,7 @@ void CollapsibleGroup::slotEffectUp()
 void CollapsibleGroup::slotEffectDown()
 {
     QList <int> indexes;
-    for (int i = 0; i < m_subWidgets.count(); i++)
+    for (int i = 0; i < m_subWidgets.count(); ++i)
         indexes << m_subWidgets.at(i)->effectIndex();
     emit changeEffectPosition(indexes, false);
 }
@@ -176,7 +176,7 @@ void CollapsibleGroup::slotSaveGroup()
     QDomDocument doc = effectsData();
     QDomElement base = doc.documentElement();
     QDomNodeList effects = base.elementsByTagName("effect");
-    for (int i = 0; i < effects.count(); i++) {
+    for (int i = 0; i < effects.count(); ++i) {
        QDomElement eff = effects.at(i).toElement();
         eff.removeAttribute("kdenlive_ix");
        EffectInfo info;
@@ -205,7 +205,7 @@ void CollapsibleGroup::slotSaveGroup()
 void CollapsibleGroup::slotResetGroup()
 {
     QMutexLocker lock(&m_mutex);
-    for (int i = 0; i < m_subWidgets.count(); i++)
+    for (int i = 0; i < m_subWidgets.count(); ++i)
         m_subWidgets.at(i)->slotResetEffect();
 }
 
@@ -337,10 +337,10 @@ void CollapsibleGroup::dropEvent(QDropEvent *event)
                return;
            }
            // Moving group
-           for (int i = 0; i < pastedEffects.count(); i++) {
+           for (int i = 0; i < pastedEffects.count(); ++i) {
                pastedEffectIndexes << pastedEffects.at(i).toElement().attribute("kdenlive_ix").toInt();
            }
-           for (int i = 0; i < m_subWidgets.count(); i++) {
+           for (int i = 0; i < m_subWidgets.count(); ++i) {
                currentEffectIndexes << m_subWidgets.at(i)->effectIndex();
            }
            kDebug()<<"PASTING: "<<pastedEffectIndexes<<" TO "<<currentEffectIndexes;
@@ -405,7 +405,7 @@ QDomDocument CollapsibleGroup::effectsData()
 
 void CollapsibleGroup::adjustEffects()
 {
-    for (int i = 0; i < m_subWidgets.count(); i++) {
+    for (int i = 0; i < m_subWidgets.count(); ++i) {
        m_subWidgets.at(i)->adjustButtons(i, m_subWidgets.count());
     }
 }