[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdeplasma-addons/applets/weather
From:       Davide Bettio <davide.bettio () kdemail ! net>
Date:       2009-03-31 20:54:28
Message-ID: 1238532868.082980.26670.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 947599 by bettio:

We don't need to delete widgets: Qt does everything for us :)


 M  +2 -18     weatherapplet.cpp  


--- trunk/KDE/kdeplasma-addons/applets/weather/weatherapplet.cpp #947598:947599
@@ -200,22 +200,6 @@
     if (m_addDialog) {
         delete m_addDialog;
     }
-
-    if (m_bottomLayout) {
-         if (m_fiveDaysView) {
-             kDebug() << "Delete the Plasma::m_fiveDaysView";
-             delete m_fiveDaysView;
-             m_fiveDaysView = 0;
-             kDebug() << "Finished deleting Plasma::m_fiveDaysView";
-         }
-
-         if (m_detailsView) {
-             kDebug() << "Delete the Plasma::m_detailsView";
-             delete m_detailsView;
-             m_detailsView = 0;
-             kDebug() << "Finished deleting Plasma::m_detailsView";
-         }
-    }
 }
 
 void WeatherApplet::toolTipAboutToShow()
@@ -646,7 +630,7 @@
     if (data["Total Weather Days"].toInt() > 0) {
         if (!m_fiveDaysView) {
             kDebug() << "Create 5 Days Plasma::WeatherView";
-            m_fiveDaysView = new Plasma::WeatherView();
+            m_fiveDaysView = new Plasma::WeatherView(m_tabBar);
             m_fiveDaysView->setHasHeader(true);
             m_fiveDaysView->setOrientation(Qt::Vertical);
         }
@@ -765,7 +749,7 @@
     // Details data
     if (!m_detailsView) {
         kDebug() << "Create Details Plasma::WeatherView";
-        m_detailsView = new Plasma::WeatherView();
+        m_detailsView = new Plasma::WeatherView(m_tabBar);
         m_detailsView->setHasHeader(false);
         m_detailsView->setOrientation(Qt::Horizontal);
     }
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic