diff options
| author | Takeshi Abe <tabe@fixedpoint.jp> | 2012-01-26 01:02:30 +0900 |
|---|---|---|
| committer | Takeshi Abe <tabe@fixedpoint.jp> | 2012-01-26 03:03:08 +0900 |
| commit | bb51e8e72be424f0fd92350006c536a97fd4b210 (patch) | |
| tree | 043f66cf4677e8fe69309be76565515e4a68f9a0 /basctl | |
| parent | 85d34beec5f8196896487c96028b2dc2ce3cf6c8 (diff) | |
Deleting a null pointer is safe
Diffstat (limited to 'basctl')
| -rw-r--r-- | basctl/source/dlged/managelang.cxx | 3 | ||||
| -rw-r--r-- | basctl/source/dlged/propbrw.cxx | 3 |
2 files changed, 2 insertions, 4 deletions
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(); |