That's safe to delete NULL.

modules/gui/skins2/controls/ctrl_list.cpp

@@ -65,10 +65,7 @@ CtrlList::~CtrlList()
 {
     m_rList.getPositionVar().delObserver( this );
     m_rList.delObserver( this );
-    if( m_pImage )
-    {
     delete m_pImage;
-    }
 }
 
 
@@ -411,10 +408,7 @@ void CtrlList::autoScroll()
 
 void CtrlList::makeImage()
 {
-    if( m_pImage )
-    {
     delete m_pImage;
-    }
 
     // Get the size of the control
     const Position *pPos = getPosition();

modules/gui/skins2/src/file_bitmap.cpp

@@ -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;
 }
 
 

modules/gui/skins2/src/ft2_bitmap.cpp

@@ -36,10 +36,7 @@ FT2Bitmap::FT2Bitmap( intf_thread_t *pIntf, int width, int height ):
 
 FT2Bitmap::~FT2Bitmap()
 {
-    if( m_pData )
-    {
     delete[] m_pData;
-    }
 }
 
 

modules/gui/skins2/src/scaled_bitmap.cpp

@@ -104,9 +104,6 @@ ScaledBitmap::ScaledBitmap( intf_thread_t *pIntf, const GenericBitmap &rBitmap,
 
 ScaledBitmap::~ScaledBitmap()
 {
-    if( m_pData )
-    {
     delete[] m_pData;
-    }
 }
 

modules/gui/skins2/utils/ustring.cpp

@@ -130,10 +130,7 @@ UString::UString( intf_thread_t *pIntf, const char *pString ):
 
 UString::~UString()
 {
-    if( m_pString )
-    {
     delete[] m_pString;
-    }
 }
 
 

modules/video_filter/atmo/AtmoConfig.cpp

@@ -211,7 +211,6 @@ int CAtmoConfig::getNumChannelAssignments() {
 void CAtmoConfig::clearChannelMappings() {
     for(int i=1;i<10;i++) {
         tChannelAssignment *ca = m_ChannelAssignments[i];
-        if(ca!=NULL)
         delete ca;
         m_ChannelAssignments[i] = NULL;
     }
@@ -220,7 +219,6 @@ void CAtmoConfig::clearChannelMappings() {
 void CAtmoConfig::clearAllChannelMappings() {
     for(int i=0;i<10;i++) {
         tChannelAssignment *ca = m_ChannelAssignments[i];
-        if(ca!=NULL)
         delete ca;
         m_ChannelAssignments[i] = NULL;
     }