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

List:       kde-commits
Subject:    KDE/kdeedu/marble/src/plugins/render/aprs
From:       Laurent Montel <montel () kde ! org>
Date:       2011-01-16 14:03:21
Message-ID: 20110116140321.1FC033E1F1 () svn ! kde ! org
[Download RAW message or body]

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
[prev in list] [next in list] [prev in thread] [next in thread]