diff options
author | Noel Grandin <noelgrandin@gmail.com> | 2015-06-13 22:15:31 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2015-06-15 14:46:41 +0200 |
commit | 4729774b244db7a175077ed0c70aa48be62bc60e (patch) | |
tree | 9f0f2e329d269835d4c20a8183a2772b734608f3 /xmloff/source/draw | |
parent | 9db80b1499e09cde000160434728f7b231c30db8 (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
Diffstat (limited to 'xmloff/source/draw')
-rw-r--r-- | xmloff/source/draw/sdxmlimp.cxx | 30 | ||||
-rw-r--r-- | xmloff/source/draw/shapeimport.cxx | 24 | ||||
-rw-r--r-- | xmloff/source/draw/ximpshap.cxx | 3 |
3 files changed, 23 insertions, 34 deletions
diff --git a/xmloff/source/draw/sdxmlimp.cxx b/xmloff/source/draw/sdxmlimp.cxx index f7df18e0938c..b38d117ca590 100644 --- a/xmloff/source/draw/sdxmlimp.cxx +++ b/xmloff/source/draw/sdxmlimp.cxx @@ -421,26 +421,16 @@ SdXMLImport::~SdXMLImport() throw () mpMasterStylesContext->ReleaseRef(); // delete all token maps - if(mpDocElemTokenMap) - delete mpDocElemTokenMap; - if(mpBodyElemTokenMap) - delete mpBodyElemTokenMap; - if(mpStylesElemTokenMap) - delete mpStylesElemTokenMap; - if(mpMasterPageElemTokenMap) - delete mpMasterPageElemTokenMap; - if(mpMasterPageAttrTokenMap) - delete mpMasterPageAttrTokenMap; - if(mpPageMasterAttrTokenMap) - delete mpPageMasterAttrTokenMap; - if(mpPageMasterStyleAttrTokenMap) - delete mpPageMasterStyleAttrTokenMap; - if(mpDrawPageAttrTokenMap) - delete mpDrawPageAttrTokenMap; - if(mpDrawPageElemTokenMap) - delete mpDrawPageElemTokenMap; - if(mpPresentationPlaceholderAttrTokenMap) - delete mpPresentationPlaceholderAttrTokenMap; + delete mpDocElemTokenMap; + delete mpBodyElemTokenMap; + delete mpStylesElemTokenMap; + delete mpMasterPageElemTokenMap; + delete mpMasterPageAttrTokenMap; + delete mpPageMasterAttrTokenMap; + delete mpPageMasterStyleAttrTokenMap; + delete mpDrawPageAttrTokenMap; + delete mpDrawPageElemTokenMap; + delete mpPresentationPlaceholderAttrTokenMap; } const SvXMLTokenMap& SdXMLImport::GetDocElemTokenMap() diff --git a/xmloff/source/draw/shapeimport.cxx b/xmloff/source/draw/shapeimport.cxx index c8dc20cade86..d86ee5094c3a 100644 --- a/xmloff/source/draw/shapeimport.cxx +++ b/xmloff/source/draw/shapeimport.cxx @@ -200,18 +200,18 @@ XMLShapeImportHelper::~XMLShapeImportHelper() mpPresPagePropsMapper = 0L; } - if(mpGroupShapeElemTokenMap) delete mpGroupShapeElemTokenMap; - if(mpFrameShapeElemTokenMap) delete mpFrameShapeElemTokenMap; - - if(mpPolygonShapeAttrTokenMap) delete mpPolygonShapeAttrTokenMap; - if(mpPathShapeAttrTokenMap) delete mpPathShapeAttrTokenMap; - if(mp3DSceneShapeElemTokenMap) delete mp3DSceneShapeElemTokenMap; - if(mp3DObjectAttrTokenMap) delete mp3DObjectAttrTokenMap; - if(mp3DPolygonBasedAttrTokenMap) delete mp3DPolygonBasedAttrTokenMap; - if(mp3DCubeObjectAttrTokenMap) delete mp3DCubeObjectAttrTokenMap; - if(mp3DSphereObjectAttrTokenMap) delete mp3DSphereObjectAttrTokenMap; - if(mp3DSceneShapeAttrTokenMap) delete mp3DSceneShapeAttrTokenMap; - if(mp3DLightAttrTokenMap) delete mp3DLightAttrTokenMap; + delete mpGroupShapeElemTokenMap; + delete mpFrameShapeElemTokenMap; + + delete mpPolygonShapeAttrTokenMap; + delete mpPathShapeAttrTokenMap; + delete mp3DSceneShapeElemTokenMap; + delete mp3DObjectAttrTokenMap; + delete mp3DPolygonBasedAttrTokenMap; + delete mp3DCubeObjectAttrTokenMap; + delete mp3DSphereObjectAttrTokenMap; + delete mp3DSceneShapeAttrTokenMap; + delete mp3DLightAttrTokenMap; // Styles or AutoStyles context? if(mpStylesContext) diff --git a/xmloff/source/draw/ximpshap.cxx b/xmloff/source/draw/ximpshap.cxx index 316b37912a00..fac5f3bcdeba 100644 --- a/xmloff/source/draw/ximpshap.cxx +++ b/xmloff/source/draw/ximpshap.cxx @@ -2574,8 +2574,7 @@ SdXMLChartShapeContext::SdXMLChartShapeContext( SdXMLChartShapeContext::~SdXMLChartShapeContext() { - if( mpChartContext ) - delete mpChartContext; + delete mpChartContext; } void SdXMLChartShapeContext::StartElement(const uno::Reference< xml::sax::XAttributeList>& xAttrList) |