From ba6cfa6067b821c9dd26c8ebc9c41b0e10efe386 Mon Sep 17 00:00:00 2001 From: Montel Laurent Date: Wed, 15 May 2013 13:44:26 +0200 Subject: [PATCH] use --i --- src/customtrackview.cpp | 4 ++-- src/docclipbase.cpp | 2 +- src/documentchecker.cpp | 4 ++-- src/dvdwizardmenu.cpp | 4 ++-- src/effectslist.cpp | 2 +- src/effectslistwidget.cpp | 2 +- src/effectstack/collapsiblegroup.cpp | 4 ++-- src/kdenlivedoc.cpp | 8 ++++---- src/kis_cubic_curve.cpp | 2 +- src/projectlist.cpp | 12 ++++++------ src/renderer.cpp | 8 ++++---- src/trackview.cpp | 8 ++++---- 12 files changed, 30 insertions(+), 30 deletions(-) diff --git a/src/customtrackview.cpp b/src/customtrackview.cpp index 72bbe7a3..649eb65a 100644 --- a/src/customtrackview.cpp +++ b/src/customtrackview.cpp @@ -367,7 +367,7 @@ int CustomTrackView::getPreviousVideoTrack(int track) { track = m_document->tracksCount() - track - 1; track --; - for (int i = track; i > -1; i--) { + for (int i = track; i > -1; --i) { if (m_document->trackInfoAt(i).type == VIDEOTRACK) return i + 1; } return 0; @@ -4895,7 +4895,7 @@ void CustomTrackView::moveGroup(QList startClip, QList sta AbstractGroupItem *group = (AbstractGroupItem*) children.at(i); if (!groupList.contains(group)) groupList.append(group); children.removeAll(children.at(i)); - i--; + --i; } } diff --git a/src/docclipbase.cpp b/src/docclipbase.cpp index b91200df..65476bd0 100644 --- a/src/docclipbase.cpp +++ b/src/docclipbase.cpp @@ -1278,7 +1278,7 @@ void DocClipBase::removeCutZone(int in, int out) for (int i = 0; i < m_cutZones.count(); ++i) { if (m_cutZones.at(i).zone == p) { m_cutZones.removeAt(i); - i--; + --i; } } } diff --git a/src/documentchecker.cpp b/src/documentchecker.cpp index d527a42b..f2b09d62 100644 --- a/src/documentchecker.cpp +++ b/src/documentchecker.cpp @@ -879,7 +879,7 @@ void DocumentChecker::slotDeleteSelected() if (deletedIds.contains(e.attribute("id"))) { // Remove clip kdenlivedoc.removeChild(e); - i--; + --i; j++; } } @@ -889,7 +889,7 @@ void DocumentChecker::slotDeleteSelected() if (deletedIds.contains(e.attribute("id").section('_', 0, 0)) || deletedIds.contains(e.attribute("id").section(':', 1, 1).section('_', 0, 0))) { // Remove clip mlt.removeChild(e); - i--; + --i; } } diff --git a/src/dvdwizardmenu.cpp b/src/dvdwizardmenu.cpp index e1c18a1e..89d20dcb 100644 --- a/src/dvdwizardmenu.cpp +++ b/src/dvdwizardmenu.cpp @@ -239,7 +239,7 @@ bool DvdWizardMenu::isComplete() const // check that we have a "Play all" entry if (targets.contains(0)) return true; // ... or that each video file has a button - for (int i = m_view.target_list->count() - 1; i > 0; i--) { + for (int i = m_view.target_list->count() - 1; i > 0; --i) { // If there is a vob file entry and it has no button assigned, don't allow to go further if (m_view.target_list->itemIcon(i).isNull() == false && !targets.contains(i)) { #if KDE_IS_VERSION(4,7,0) @@ -813,7 +813,7 @@ void DvdWizardMenu::loadXml(DVDFORMAT format, QDomElement xml) for (int i = 0; i < list.count(); ++i) { if (list.at(i)->type() == DvdButtonItem) { delete list.at(i); - i--; + --i; } } } diff --git a/src/effectslist.cpp b/src/effectslist.cpp index fe7078fb..95ddca8a 100644 --- a/src/effectslist.cpp +++ b/src/effectslist.cpp @@ -318,7 +318,7 @@ void EffectsList::removeMetaProperties(QDomElement producer) QDomElement e = params.item(i).toElement(); if (e.attribute("name").startsWith("meta")) { producer.removeChild(params.item(i)); - i--; + --i; } } } diff --git a/src/effectslistwidget.cpp b/src/effectslistwidget.cpp index f2762673..01ef3ed6 100644 --- a/src/effectslistwidget.cpp +++ b/src/effectslistwidget.cpp @@ -101,7 +101,7 @@ void EffectsListWidget::initList(QMenu *effectsMenu, KActionCategory *effectActi QString currentName = topLevelItem(i)->text(0); if (currentName != i18n("Misc") && currentName != i18n("Audio") && currentName != i18nc("Folder Name", "Custom") && !folderNames.contains(currentName)) { takeTopLevelItem(i); - i--; + --i; } } diff --git a/src/effectstack/collapsiblegroup.cpp b/src/effectstack/collapsiblegroup.cpp index 7346b8fe..c4824e9f 100644 --- a/src/effectstack/collapsiblegroup.cpp +++ b/src/effectstack/collapsiblegroup.cpp @@ -144,7 +144,7 @@ void CollapsibleGroup::slotDeleteGroup() QDomDocument doc; // delete effects from the last one to the first, otherwise each deletion would trigger an update // in other effects's kdenlive_ix index. - for (int i = m_subWidgets.count() - 1; i >= 0; i--) + for (int i = m_subWidgets.count() - 1; i >= 0; --i) doc.appendChild(doc.importNode(m_subWidgets.at(i)->effect(), true)); emit deleteGroup(doc); } @@ -261,7 +261,7 @@ void CollapsibleGroup::removeGroup(int ix, QVBoxLayout *layout) QMutexLocker lock(&m_mutex); QVBoxLayout *vbox = static_cast(widgetFrame->layout()); if (vbox == NULL) return; - for (int i = m_subWidgets.count() - 1; i >= 0 ; i--) { + for (int i = m_subWidgets.count() - 1; i >= 0 ; --i) { vbox->removeWidget(m_subWidgets.at(i)); layout->insertWidget(ix, m_subWidgets.at(i)); m_subWidgets.at(i)->decoframe->setObjectName("decoframe"); diff --git a/src/kdenlivedoc.cpp b/src/kdenlivedoc.cpp index e3fe2161..d648261d 100644 --- a/src/kdenlivedoc.cpp +++ b/src/kdenlivedoc.cpp @@ -1797,28 +1797,28 @@ void KdenliveDoc::cleanupBackupFiles() for (int i = 0; i < hourList.count(); i += step) { kDebug()<<"REMOVE AT: "< 20) { int step = dayList.count() / 10; for (int i = 0; i < dayList.count(); i += step) { dayList.removeAt(i); - i--; + --i; } } else dayList.clear(); if (weekList.count() > 20) { int step = weekList.count() / 10; for (int i = 0; i < weekList.count(); i += step) { weekList.removeAt(i); - i--; + --i; } } else weekList.clear(); if (oldList.count() > 20) { int step = oldList.count() / 10; for (int i = 0; i < oldList.count(); i += step) { oldList.removeAt(i); - i--; + --i; } } else oldList.clear(); diff --git a/src/kis_cubic_curve.cpp b/src/kis_cubic_curve.cpp index 52488a62..128cde9b 100644 --- a/src/kis_cubic_curve.cpp +++ b/src/kis_cubic_curve.cpp @@ -94,7 +94,7 @@ public: / (b.last() + a.last() * alpha.last()); - for (i = size - 2; i >= 0; i--) + for (i = size - 2; i >= 0; --i) x[i] = alpha[i+1] * x[i+1] + beta[i+1]; return x; diff --git a/src/projectlist.cpp b/src/projectlist.cpp index 86b6fa67..e2940eaf 100644 --- a/src/projectlist.cpp +++ b/src/projectlist.cpp @@ -376,23 +376,23 @@ void ProjectList::setupMenu(QMenu *addMenu, QAction *defaultAction) if (actions.at(i)->data().toString() == "clip_properties") { m_editButton->setDefaultAction(actions.at(i)); actions.removeAt(i); - i--; + --i; } else if (actions.at(i)->data().toString() == "delete_clip") { m_deleteButton->setDefaultAction(actions.at(i)); actions.removeAt(i); - i--; + --i; } else if (actions.at(i)->data().toString() == "edit_clip") { m_openAction = actions.at(i); actions.removeAt(i); - i--; + --i; } else if (actions.at(i)->data().toString() == "reload_clip") { m_reloadAction = actions.at(i); actions.removeAt(i); - i--; + --i; } else if (actions.at(i)->data().toString() == "proxy_clip") { m_proxyAction = actions.at(i); actions.removeAt(i); - i--; + --i; } } @@ -3065,7 +3065,7 @@ void ProjectList::slotCheckJobProcess() // remove finished jobs AbstractClipJob *job = m_jobList.takeAt(i); job->deleteLater(); - i--; + --i; } } emit jobCount(count); diff --git a/src/renderer.cpp b/src/renderer.cpp index 8becf375..be5edc3d 100644 --- a/src/renderer.cpp +++ b/src/renderer.cpp @@ -3954,7 +3954,7 @@ void Render::mltPlantTransition(Mlt::Field *field, Mlt::Transition &tr, int a_tr field->plant_transition(tr, a_track, b_track); // re-add upper transitions - for (int i = trList.count() - 1; i >= 0; i--) { + for (int i = trList.count() - 1; i >= 0; --i) { //kDebug()<< "REPLANT ON TK: "<get_a_track()<<", "<get_b_track(); field->plant_transition(*trList.at(i), trList.at(i)->get_a_track(), trList.at(i)->get_b_track()); } @@ -4478,7 +4478,7 @@ QList Render::mltInsertTrack(int ix, bool videoTrack) mltPlantTransition(field, transition, 1, ct); // re-add transitions - for (int i = trList.count() - 1; i >= 0; i--) { + for (int i = trList.count() - 1; i >= 0; --i) { field->plant_transition(*trList.at(i), trList.at(i)->get_a_track(), trList.at(i)->get_b_track()); } qDeleteAll(trList); @@ -4507,7 +4507,7 @@ void Render::mltDeleteTrack(int ix) } if (mappedProps.value("mlt_service") == "mix" && mappedProps.value("b_track").toInt() == tracksCount) { tractor.removeChild(transitions.at(i)); - i--; + --i; } else if (mappedProps.value("mlt_service") != "mix" && (mappedProps.value("b_track").toInt() >= ix || mappedProps.value("a_track").toInt() >= ix)) { // Transition needs to be moved int a_track = mappedProps.value("a_track").toInt(); @@ -4516,7 +4516,7 @@ void Render::mltDeleteTrack(int ix) if (b_track == ix) { // transition was on the deleted track, so remove it tractor.removeChild(transitions.at(i)); - i--; + --i; continue; } if (b_track > 0 && b_track > ix) b_track --; diff --git a/src/trackview.cpp b/src/trackview.cpp index 2f045001..2d2c3363 100644 --- a/src/trackview.cpp +++ b/src/trackview.cpp @@ -225,7 +225,7 @@ void TrackView::parseDocument(const QDomDocument &doc) m_invalidProducers.append(id); m_documentErrors.append(i18n("Invalid clip producer %1\n", id)); doc.documentElement().removeChild(producers.at(i)); - i--; + --i; } } @@ -322,7 +322,7 @@ void TrackView::parseDocument(const QDomDocument &doc) // invalid transition, remove it m_documentErrors.append(i18n("Removed invalid transition: %1", e.attribute("id")) + '\n'); tractor.removeChild(transitions.item(i)); - i--; + --i; continue; } if (transitionAdd || mlt_service != "mix") { @@ -398,7 +398,7 @@ void TrackView::parseDocument(const QDomDocument &doc) m_documentErrors.append(i18n("Removed invalid transition: (%1, %2, %3)", e.attribute("id"), mlt_service, transitionId) + '\n'); kDebug() << "///// REMOVED INVALID TRANSITION: " << e.attribute("id"); tractor.removeChild(transitions.item(i)); - i--; + --i; } else if (m_trackview->canBePastedTo(transitionInfo, TRANSITIONWIDGET)) { Transition *tr = new Transition(transitionInfo, a_track, m_doc->fps(), base, isAutomatic); if (forceTrack) tr->setForcedTrack(true, a_track); @@ -410,7 +410,7 @@ void TrackView::parseDocument(const QDomDocument &doc) else { m_documentErrors.append(i18n("Removed overlapping transition: (%1, %2, %3)", e.attribute("id"), mlt_service, transitionId) + '\n'); tractor.removeChild(transitions.item(i)); - i--; + --i; } } } -- 2.39.2