diff options
author | Jochen Nitschke <j.nitschke+logerrit@ok.de> | 2017-01-23 09:01:42 +0100 |
---|---|---|
committer | Jochen Nitschke <j.nitschke+logerrit@ok.de> | 2017-01-23 13:50:45 +0000 |
commit | abc8057ab1a1189ff7f88d42b13b363ceb228d16 (patch) | |
tree | 20145fa90c9a1bf91dea3489d384fe6d28ac2ec0 /svx/source/svdraw/svdhdl.cxx | |
parent | 5f1bc19a9a0dbdf88e11ef81cc8c2f302748632c (diff) |
pass ownership by pointer
to show the intend of the code.
assert on invalid objects.
remove useless OSL_ENSURE.
make sure list of owning pointers can't be copied.
Change-Id: I3cccf8ea87585ec9fe62921203a4d12e617ce15c
Reviewed-on: https://gerrit.libreoffice.org/33393
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
Diffstat (limited to 'svx/source/svdraw/svdhdl.cxx')
-rw-r--r-- | svx/source/svdraw/svdhdl.cxx | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/svx/source/svdraw/svdhdl.cxx b/svx/source/svdraw/svdhdl.cxx index 2f8a46854969..0b51b7ccf9f9 100644 --- a/svx/source/svdraw/svdhdl.cxx +++ b/svx/source/svdraw/svdhdl.cxx @@ -630,7 +630,7 @@ void SdrHdl::CreateB2dIAObject() if (pNewOverlayObject) { xManager->add(*pNewOverlayObject); - maOverlayGroup.append(*pNewOverlayObject); + maOverlayGroup.append(pNewOverlayObject); } } } @@ -1142,7 +1142,7 @@ void SdrHdlColor::CreateB2dIAObject() // OVERLAYMANAGER xManager->add(*pNewOverlayObject); - maOverlayGroup.append(*pNewOverlayObject); + maOverlayGroup.append(pNewOverlayObject); } } } @@ -1307,7 +1307,7 @@ void SdrHdlGradient::CreateB2dIAObject() pNewOverlayObject->setBaseColor(IsGradient() ? Color(COL_BLACK) : Color(COL_BLUE)); xManager->add(*pNewOverlayObject); - maOverlayGroup.append(*pNewOverlayObject); + maOverlayGroup.append(pNewOverlayObject); // arrowhead Point aLeft(aMidPoint.X() + (sal_Int32)(aPerpend.getX() * fHalfArrowWidth), @@ -1329,7 +1329,7 @@ void SdrHdlGradient::CreateB2dIAObject() DBG_ASSERT(pNewOverlayObject, "Got NO new IAO!"); xManager->add(*pNewOverlayObject); - maOverlayGroup.append(*pNewOverlayObject); + maOverlayGroup.append(pNewOverlayObject); } } } @@ -1457,7 +1457,7 @@ void SdrHdlLine::CreateB2dIAObject() pNewOverlayObject->setBaseColor(Color(COL_LIGHTRED)); xManager->add(*pNewOverlayObject); - maOverlayGroup.append(*pNewOverlayObject); + maOverlayGroup.append(pNewOverlayObject); } } } @@ -1517,7 +1517,7 @@ void SdrHdlBezWgt::CreateB2dIAObject() pNewOverlayObject->setBaseColor(Color(COL_LIGHTBLUE)); xManager->add(*pNewOverlayObject); - maOverlayGroup.append(*pNewOverlayObject); + maOverlayGroup.append(pNewOverlayObject); } } } @@ -1563,7 +1563,7 @@ void E3dVolumeMarker::CreateB2dIAObject() pNewOverlayObject->setBaseColor(Color(COL_BLACK)); xManager->add(*pNewOverlayObject); - maOverlayGroup.append(*pNewOverlayObject); + maOverlayGroup.append(pNewOverlayObject); } } } @@ -1629,7 +1629,7 @@ void ImpEdgeHdl::CreateB2dIAObject() if (pNewOverlayObject) { xManager->add(*pNewOverlayObject); - maOverlayGroup.append(*pNewOverlayObject); + maOverlayGroup.append(pNewOverlayObject); } } } @@ -1750,7 +1750,7 @@ void ImpMeasureHdl::CreateB2dIAObject() if (pNewOverlayObject) { xManager->add(*pNewOverlayObject); - maOverlayGroup.append(*pNewOverlayObject); + maOverlayGroup.append(pNewOverlayObject); } } } @@ -1821,7 +1821,7 @@ void ImpTextframeHdl::CreateB2dIAObject() // OVERLAYMANAGER pNewOverlayObject->setHittable(false); xManager->add(*pNewOverlayObject); - maOverlayGroup.append(*pNewOverlayObject); + maOverlayGroup.append(pNewOverlayObject); } } } @@ -2437,7 +2437,7 @@ void SdrCropHdl::CreateB2dIAObject() if(pOverlayObject) { xManager->add(*pOverlayObject); - maOverlayGroup.append(*pOverlayObject); + maOverlayGroup.append(pOverlayObject); } } } @@ -2651,7 +2651,7 @@ void SdrCropViewHdl::CreateB2dIAObject() pNew->setHittable(false); xManager->add(*pNew); - maOverlayGroup.append(*pNew); + maOverlayGroup.append(pNew); } } } |