That's safe to delete NULL.
authorRémi Duraffort <ivoire@videolan.org>
Thu, 16 Jul 2009 08:13:00 +0000 (10:13 +0200)
committerRémi Duraffort <ivoire@videolan.org>
Thu, 16 Jul 2009 08:14:33 +0000 (10:14 +0200)
modules/gui/skins2/controls/ctrl_list.cpp
modules/gui/skins2/src/file_bitmap.cpp
modules/gui/skins2/src/ft2_bitmap.cpp
modules/gui/skins2/src/scaled_bitmap.cpp
modules/gui/skins2/utils/ustring.cpp
modules/video_filter/atmo/AtmoConfig.cpp

index 7bf3feb..c0dc427 100644 (file)
@@ -65,10 +65,7 @@ CtrlList::~CtrlList()
 {
     m_rList.getPositionVar().delObserver( this );
     m_rList.delObserver( this );
-    if( m_pImage )
-    {
-        delete m_pImage;
-    }
+    delete m_pImage;
 }
 
 
@@ -411,10 +408,7 @@ void CtrlList::autoScroll()
 
 void CtrlList::makeImage()
 {
-    if( m_pImage )
-    {
-        delete m_pImage;
-    }
+    delete m_pImage;
 
     // Get the size of the control
     const Position *pPos = getPosition();
index 5ab377d..de23e8d 100644 (file)
@@ -85,7 +85,7 @@ FileBitmap::FileBitmap( intf_thread_t *pIntf, image_handler_t *pImageHandler,
 
 FileBitmap::~FileBitmap()
 {
-    if( m_pData ) delete[] m_pData;
+    delete[] m_pData;
 }
 
 
index 7cc8a84..e743755 100644 (file)
@@ -36,10 +36,7 @@ FT2Bitmap::FT2Bitmap( intf_thread_t *pIntf, int width, int height ):
 
 FT2Bitmap::~FT2Bitmap()
 {
-    if( m_pData )
-    {
-        delete[] m_pData;
-    }
+    delete[] m_pData;
 }
 
 
index 576c96f..0491399 100644 (file)
@@ -104,9 +104,6 @@ ScaledBitmap::ScaledBitmap( intf_thread_t *pIntf, const GenericBitmap &rBitmap,
 
 ScaledBitmap::~ScaledBitmap()
 {
-    if( m_pData )
-    {
-        delete[] m_pData;
-    }
+    delete[] m_pData;
 }
 
index 76fb03c..3f1f869 100644 (file)
@@ -130,10 +130,7 @@ UString::UString( intf_thread_t *pIntf, const char *pString ):
 
 UString::~UString()
 {
-    if( m_pString )
-    {
-        delete[] m_pString;
-    }
+    delete[] m_pString;
 }
 
 
index 5e1fdf7..1249beb 100644 (file)
@@ -211,8 +211,7 @@ int CAtmoConfig::getNumChannelAssignments() {
 void CAtmoConfig::clearChannelMappings() {
     for(int i=1;i<10;i++) {
         tChannelAssignment *ca = m_ChannelAssignments[i];
-        if(ca!=NULL)
-           delete ca;
+        delete ca;
         m_ChannelAssignments[i] = NULL;
     }
 }
@@ -220,8 +219,7 @@ void CAtmoConfig::clearChannelMappings() {
 void CAtmoConfig::clearAllChannelMappings() {
     for(int i=0;i<10;i++) {
         tChannelAssignment *ca = m_ChannelAssignments[i];
-        if(ca!=NULL)
-           delete ca;
+        delete ca;
         m_ChannelAssignments[i] = NULL;
     }
 }