diff options
author | Noel Grandin <noelgrandin@gmail.com> | 2015-06-13 22:15:31 +0200 |
---|---|---|
committer | Ashod Nakashian <ashod.nakashian@collabora.co.uk> | 2016-02-06 09:12:18 -0500 |
commit | 8ebcaeb22f6fe05750b50631dc67299b2f78e4e3 (patch) | |
tree | 4746f4715e61b195d5b0b8758a0a4bca5058dfa3 /chart2 | |
parent | a9609b568ce74fa426d3322dd9270eba9c4d038a (diff) |
remove unnecessary check for null when calling delete
Idea originally from caolan.
Found using the following command:
find . -name *.cxx | xargs /opt/local/bin/grep -zlP '(?m)if\s*\(\s*\w+\s*\)\s*delete\s+\w+\;'
Change-Id: I3338f4e22193a6dfd6219c8c75835224a3392763
(cherry picked from commit 4729774b244db7a175077ed0c70aa48be62bc60e)
Diffstat (limited to 'chart2')
-rw-r--r-- | chart2/source/controller/accessibility/AccessibleTextHelper.cxx | 3 | ||||
-rw-r--r-- | chart2/source/controller/drawinglayer/ViewElementListProvider.cxx | 3 |
2 files changed, 2 insertions, 4 deletions
diff --git a/chart2/source/controller/accessibility/AccessibleTextHelper.cxx b/chart2/source/controller/accessibility/AccessibleTextHelper.cxx index 878ecd1914c4..f2321666a279 100644 --- a/chart2/source/controller/accessibility/AccessibleTextHelper.cxx +++ b/chart2/source/controller/accessibility/AccessibleTextHelper.cxx @@ -53,8 +53,7 @@ AccessibleTextHelper::AccessibleTextHelper( AccessibleTextHelper::~AccessibleTextHelper() { - if( m_pTextHelper ) - delete m_pTextHelper; + delete m_pTextHelper; } // ____ XInitialization ____ diff --git a/chart2/source/controller/drawinglayer/ViewElementListProvider.cxx b/chart2/source/controller/drawinglayer/ViewElementListProvider.cxx index 9f60c63bb405..4dbe04c2f4e3 100644 --- a/chart2/source/controller/drawinglayer/ViewElementListProvider.cxx +++ b/chart2/source/controller/drawinglayer/ViewElementListProvider.cxx @@ -54,8 +54,7 @@ ViewElementListProvider::ViewElementListProvider( DrawModelWrapper* pDrawModelWr ViewElementListProvider::~ViewElementListProvider() { - if(m_pFontList) - delete m_pFontList; + delete m_pFontList; } XColorListRef ViewElementListProvider::GetColorTable() const |