[Marble-commits] KDE/kdeedu/marble/src/plugins/render/aprs

Laurent Montel montel at kde.org
Sun Jan 16 15:03:21 CET 2011


SVN commit 1214818 by mlaurent:

don't check pointer before to delete it


 M  +0 -1      AprsObject.cpp  
 M  +0 -2      AprsPlugin.cpp  
 M  +0 -2      posix_qextserialport.cpp  


--- trunk/KDE/kdeedu/marble/src/plugins/render/aprs/AprsObject.cpp #1214817:1214818
@@ -43,7 +43,6 @@
 
 AprsObject::~AprsObject()
 {
-    if ( m_pixmap )
         delete m_pixmap;
 }
 
--- trunk/KDE/kdeedu/marble/src/plugins/render/aprs/AprsPlugin.cpp #1214817:1214818
@@ -65,9 +65,7 @@
 {
     stopGatherers();
 
-    if ( m_configDialog )
         delete m_configDialog;
-    if ( ui_configWidget )
         delete ui_configWidget;
 
     QMap<QString, AprsObject *>::Iterator obj;
--- trunk/KDE/kdeedu/marble/src/plugins/render/aprs/posix_qextserialport.cpp #1214817:1214818
@@ -766,12 +766,10 @@
         QIODevice::close();	// Flag the device as closed
         // QIODevice::close() doesn't actually close the port, so do that here
         ::close(fd);
-        if(readNotifier) {
             delete readNotifier;
             readNotifier = 0;
         }
     }
-}
 
 /*!
 Flushes all pending I/O to the serial port.  This function has no effect if the serial port


More information about the Marble-commits mailing list