From: Peter Penz <peter.penz19@gmail.com>
Date: Sat, 20 Jun 2009 16:49:04 +0000 (+0000)
Subject: - Don't keep pointers to objects that are obtained and deleted by other objects.
X-Git-Url: https://cloud.milkyroute.net/gitweb/dolphin.git/commitdiff_plain/419ad21309a69e5047624c09941c5d99d0be93c0

- Don't keep pointers to objects that are obtained and deleted by other objects.
- Simplify the API so that it is assured that no memory leaks are given.

svn path=/trunk/KDE/kdebase/apps/; revision=984375
---

diff --git a/src/tooltips/tooltipmanager.cpp b/src/tooltips/tooltipmanager.cpp
index 67fbf2946..b647241bf 100644
--- a/src/tooltips/tooltipmanager.cpp
+++ b/src/tooltips/tooltipmanager.cpp
@@ -54,7 +54,6 @@ ToolTipManager::ToolTipManager(QAbstractItemView* parent,
     m_generatingPreview(false),
     m_previewIsLate(false),
     m_previewPass(0),
-    m_emptyRenderedKToolTipItem(0),
     m_pix()
 {
     KToolTip::setToolTipDelegate(g_delegate);
@@ -152,14 +151,10 @@ void ToolTipManager::prepareToolTip()
         if (m_previewPass == 1) {
             // We waited 250msec and the preview is still not finished,
             // so show the toolTip with a transparent image of maximal width.
-            // When the preview finishes, m_previewIsLate will cause
-            // a direct update of the tooltip, via m_emptyRenderedKToolTipItem.
             QPixmap paddedImage(QSize(PREVIEW_WIDTH, 32));
             m_previewIsLate = true;
             paddedImage.fill(Qt::transparent);
-            KToolTipItem* toolTip = new KToolTipItem(paddedImage, m_item.getToolTipText());
-            m_emptyRenderedKToolTipItem = toolTip; // make toolTip accessible everywhere
-            showToolTip(toolTip);
+            showToolTip(paddedImage, m_item.getToolTipText());
         }
 
         ++m_previewPass;
@@ -169,9 +164,7 @@ void ToolTipManager::prepareToolTip()
         if (m_preview && m_previewIsLate) {
             // We got a preview, but it is late, the tooltip has already been shown.
             // So update the tooltip directly.
-            if (m_emptyRenderedKToolTipItem != 0) {
-                m_emptyRenderedKToolTipItem->setData(Qt::DecorationRole, KIcon(m_pix));
-            }
+            showToolTip(m_pix, m_item.getToolTipText());
             return;
         }
 
@@ -185,21 +178,18 @@ void ToolTipManager::prepareToolTip()
             icon = KIcon(KIcon(m_item.iconName()).pixmap(ICON_WIDTH, ICON_HEIGHT));
         }
 
-        KToolTipItem* toolTip = new KToolTipItem(icon, m_item.getToolTipText());
-        showToolTip(toolTip);
+        showToolTip(icon, m_item.getToolTipText());
     }
 }
 
-void ToolTipManager::showToolTip(KToolTipItem* tip)
+void ToolTipManager::showToolTip(const QIcon& icon, const QString& text)
 {
     if (QApplication::mouseButtons() & Qt::LeftButton) {
-        delete tip;
-        tip = 0;
-        // m_emptyRenderedKToolTipItem is an alias for tip.
-        m_emptyRenderedKToolTipItem = 0;
         return;
     }
 
+    KToolTipItem* tip = new KToolTipItem(icon, m_item.getToolTipText());
+
     KStyleOptionToolTip option;
     // TODO: get option content from KToolTip or add KToolTip::sizeHint() method
     option.direction      = QApplication::layoutDirection();
@@ -254,6 +244,7 @@ void ToolTipManager::showToolTip(KToolTipItem* tip)
         y = desktop.bottom() - size.height();
     }
 
+    // the ownership of tip is transferred to KToolTip
     KToolTip::showTip(QPoint(x, y), tip);
 }
 
diff --git a/src/tooltips/tooltipmanager.h b/src/tooltips/tooltipmanager.h
index 96df4819e..5cd01e820 100644
--- a/src/tooltips/tooltipmanager.h
+++ b/src/tooltips/tooltipmanager.h
@@ -68,7 +68,7 @@ private slots:
     void previewFailed(const KFileItem& item);
 
 private:
-    void showToolTip(KToolTipItem* tip);
+    void showToolTip(const QIcon& icon, const QString& text);
 
     QAbstractItemView* m_view;
     DolphinModel* m_dolphinModel;
@@ -83,9 +83,7 @@ private:
     bool m_generatingPreview;
     bool m_previewIsLate;
     int m_previewPass;
-    KToolTipItem* m_emptyRenderedKToolTipItem;
     QPixmap m_pix;
-
 };
 
 #endif