diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-09-20 09:48:50 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-09-21 10:52:46 +0200 |
| commit | ffe84f49270e0a8818365ca7d1fd5242abff5562 (patch) | |
| tree | 9fc133fc58737afcfbd49242528811eb320705c3 /sw/source | |
| parent | 6b28441fb8823ba28a9db6cf1d97aa277cdcd7b4 (diff) | |
remove SdrObject::GetHdl in favour of AddToHdlList
the code already says that using GetHdl is inefficient, and client code
should be using AddToHdlList, so just drop the bad one
Change-Id: I88c43154c8cc0988127b9292e4cda28917f54eb9
Reviewed-on: https://gerrit.libreoffice.org/60792
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sw/source')
| -rw-r--r-- | sw/source/core/draw/dcontact.cxx | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/sw/source/core/draw/dcontact.cxx b/sw/source/core/draw/dcontact.cxx index c3bb4ae61025..2f5da1bb4e36 100644 --- a/sw/source/core/draw/dcontact.cxx +++ b/sw/source/core/draw/dcontact.cxx @@ -2339,21 +2339,19 @@ basegfx::B2DPolyPolygon SwDrawVirtObj::TakeContour() const return aRetval; } -SdrHdl* SwDrawVirtObj::GetHdl(sal_uInt32 nHdlNum) const +void SwDrawVirtObj::AddToHdlList(SdrHdlList& rHdlList) const { - SdrHdl* pHdl = rRefObj.GetHdl(nHdlNum); + SdrHdlList tmpList(nullptr); + rRefObj.AddToHdlList(tmpList); - if(pHdl) + size_t cnt = tmpList.GetHdlCount(); + for(size_t i=0; i < cnt; ++i) { + SdrHdl* pHdl = tmpList.GetHdl(i); Point aP(pHdl->GetPos() + GetOffset()); pHdl->SetPos(aP); } - else - { - OSL_ENSURE(false, "Got no SdrHdl(!)"); - } - - return pHdl; + tmpList.MoveTo(rHdlList); } void SwDrawVirtObj::NbcMove(const Size& rSiz) |