diff options
author | Tomaž Vajngerl <tomaz.vajngerl@collabora.co.uk> | 2020-10-02 12:02:42 +0200 |
---|---|---|
committer | Tomaž Vajngerl <quikee@gmail.com> | 2020-10-08 16:28:29 +0200 |
commit | b7f3d35ecd63b503fb2fb9685b5475cef63fcbea (patch) | |
tree | 606485283d8aee0bc321ec253e1bcd7b00396de0 | |
parent | ca84daab26d4fec1b43be81478ef629f8c1be7cf (diff) |
rearrange to not use pUndoObject after std::move
pGeoUndo is always nullptr in this case because of std::move, so
the SetSkipChangeLayout is never set.
Issue since: https://gerrit.libreoffice.org/c/core/+/78201
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103827
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Tested-by: Tomaž Vajngerl <quikee@gmail.com>
(cherry picked from commit e31474d71235bccee8568c4a952a8319bddf39e0)
Change-Id: I4f1dcb3a25adeb0737e03aa9e9dfb193969250b6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103894
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
-rw-r--r-- | svx/source/table/tablecontroller.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/svx/source/table/tablecontroller.cxx b/svx/source/table/tablecontroller.cxx index d6541073be6e..232ed782414a 100644 --- a/svx/source/table/tablecontroller.cxx +++ b/svx/source/table/tablecontroller.cxx @@ -1529,11 +1529,11 @@ void SvxTableController::changeTableEdge(const SfxRequest& rReq) { auto pUndoObject = rModel.GetSdrUndoFactory().CreateUndoGeoObject(rTableObj); rModel.BegUndo(pUndoObject->GetComment()); - rModel.AddUndo(std::move(pUndoObject)); auto* pGeoUndo = static_cast<SdrUndoGeoObj*>(pUndoObject.get()); if (pGeoUndo) pGeoUndo->SetSkipChangeLayout(true); + rModel.AddUndo(std::move(pUndoObject)); } tools::Rectangle aBoundRect; if (rTableObj.GetUserCall()) |