From bb51e8e72be424f0fd92350006c536a97fd4b210 Mon Sep 17 00:00:00 2001
From: Takeshi Abe <tabe@fixedpoint.jp>
Date: Thu, 26 Jan 2012 01:02:30 +0900
Subject: Deleting a null pointer is safe

---
 basctl/source/dlged/managelang.cxx | 3 +--
 basctl/source/dlged/propbrw.cxx    | 3 +--
 2 files changed, 2 insertions(+), 4 deletions(-)

(limited to 'basctl/source')

diff --git a/basctl/source/dlged/managelang.cxx b/basctl/source/dlged/managelang.cxx
index eb16d60cab63..58fd9484d25e 100644
--- a/basctl/source/dlged/managelang.cxx
+++ b/basctl/source/dlged/managelang.cxx
@@ -197,8 +197,7 @@ void ManageLanguageDialog::ClearLanguageBox()
     for ( i = 0; i < nCount; ++i )
     {
         LanguageEntry* pEntry = (LanguageEntry*)( m_aLanguageLB.GetEntryData(i) );
-        if ( pEntry )
-            delete pEntry;
+        delete pEntry;
     }
     m_aLanguageLB.Clear();
 }
diff --git a/basctl/source/dlged/propbrw.cxx b/basctl/source/dlged/propbrw.cxx
index 5ee9c2886289..4e1f90f1f1f6 100644
--- a/basctl/source/dlged/propbrw.cxx
+++ b/basctl/source/dlged/propbrw.cxx
@@ -308,8 +308,7 @@ Sequence< Reference< XInterface > >
             // next element
             pCurrent = pGroupIterator && pGroupIterator->IsMore() ? pGroupIterator->Next() : NULL;
         }
-        if (pGroupIterator)
-            delete pGroupIterator;
+        delete pGroupIterator;
     }
 
     sal_Int32 nCount = aInterfaces.size();
-- 
cgit