summaryrefslogtreecommitdiff
path: root/xmloff/source/draw
diff options
context:
space:
mode:
authorNoel Grandin <noelgrandin@gmail.com>2015-06-13 22:15:31 +0200
committerNoel Grandin <noel@peralex.com>2015-06-15 14:46:41 +0200
commit4729774b244db7a175077ed0c70aa48be62bc60e (patch)
tree9f0f2e329d269835d4c20a8183a2772b734608f3 /xmloff/source/draw
parent9db80b1499e09cde000160434728f7b231c30db8 (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.cxx30
-rw-r--r--xmloff/source/draw/shapeimport.cxx24
-rw-r--r--xmloff/source/draw/ximpshap.cxx3
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)