From d67cf79fe91cb8a0d9b31a2800f68697b79983b3 Mon Sep 17 00:00:00 2001
From: Ray Lehtiniemi <rayl@mail.com>
Date: Fri, 10 Apr 2009 15:05:39 +0000
Subject: [PATCH] [PATCH 4/6] Simplify some object deletion code

Signed-off-by: Ray Lehtiniemi <rayl@mail.com>

svn path=/trunk/kdenlive/; revision=3279
---
 src/clipmanager.cpp     | 4 +---
 src/customtrackview.cpp | 8 +++-----
 src/dvdwizardvob.cpp    | 3 +--
 src/effectstackedit.cpp | 4 +---
 src/trackview.cpp       | 3 +--
 5 files changed, 7 insertions(+), 15 deletions(-)

diff --git a/src/clipmanager.cpp b/src/clipmanager.cpp
index b591d8f0..4dee9379 100644
--- a/src/clipmanager.cpp
+++ b/src/clipmanager.cpp
@@ -157,9 +157,7 @@ void ClipManager::deleteClip(const QString &clipId)
 {
     for (int i = 0; i < m_clipList.count(); i++) {
         if (m_clipList.at(i)->getId() == clipId) {
-            DocClipBase *clip = m_clipList.takeAt(i);
-            delete clip;
-            clip = NULL;
+            delete m_clipList.takeAt(i);
             break;
         }
     }
diff --git a/src/customtrackview.cpp b/src/customtrackview.cpp
index bd6f955e..91b21204 100644
--- a/src/customtrackview.cpp
+++ b/src/customtrackview.cpp
@@ -3480,8 +3480,7 @@ void CustomTrackView::editGuide(const GenTime oldPos, const GenTime pos, const Q
         bool found = false;
         for (int i = 0; i < m_guides.count(); i++) {
             if (m_guides.at(i)->position() == oldPos) {
-                Guide *item = m_guides.takeAt(i);
-                delete item;
+                delete m_guides.takeAt(i);
                 found = true;
                 break;
             }
@@ -3693,9 +3692,8 @@ void CustomTrackView::clearSearchStrings()
 
 void CustomTrackView::copyClip()
 {
-    while (m_copiedItems.count() > 0) {
-        delete m_copiedItems.takeFirst();
-    }
+    qDeleteAll(m_copiedItems);
+    m_copiedItems.clear();
     QList<QGraphicsItem *> itemList = scene()->selectedItems();
     if (itemList.count() == 0) {
         emit displayMessage(i18n("Select a clip before copying"), ErrorMessage);
diff --git a/src/dvdwizardvob.cpp b/src/dvdwizardvob.cpp
index 6955eb69..a9dbfc67 100644
--- a/src/dvdwizardvob.cpp
+++ b/src/dvdwizardvob.cpp
@@ -108,8 +108,7 @@ void DvdWizardVob::slotCheckVobList(const QString &text)
     } else if (text.isEmpty()) {
         if (allUrls.at(count - 1)->url().path().isEmpty() && allUrls.at(count - 2)->url().path().isEmpty()) {
             // The last 2 urlrequesters are empty, remove last one
-            KUrlRequester *vob = allUrls.takeLast();
-            delete vob;
+            delete allUrls.takeLast();
         }
     } else {
         if (allUrls.at(count - 1)->url().path().isEmpty()) return;
diff --git a/src/effectstackedit.cpp b/src/effectstackedit.cpp
index 8d199bba..641f82aa 100644
--- a/src/effectstackedit.cpp
+++ b/src/effectstackedit.cpp
@@ -476,9 +476,7 @@ void EffectStackEdit::slotSliderMoved(int)
 void EffectStackEdit::clearAllItems()
 {
     qDeleteAll(items);
-    foreach(UiItem *p, uiItems) {
-        delete p;
-    }
+    qDeleteAll(uiItems);
     uiItems.clear();
     items.clear();
     valueItems.clear();
diff --git a/src/trackview.cpp b/src/trackview.cpp
index 6e193b37..65989126 100644
--- a/src/trackview.cpp
+++ b/src/trackview.cpp
@@ -404,8 +404,7 @@ void TrackView::slotRebuildTrackHeaders()
 {
     QList <TrackInfo> list = m_doc->tracksList();
     QList<HeaderTrack *> widgets = findChildren<HeaderTrack *>();
-    for (int i = 0; i < widgets.count(); i++)
-        delete widgets.at(i);
+    qDeleteAll(widgets);
     int max = list.count();
     for (int i = 0; i < max; i++) {
         HeaderTrack *header = new HeaderTrack(i, list.at(max - i - 1), this);
-- 
2.39.2