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-02 16:05:56 +0200 |
| commit | e31474d71235bccee8568c4a952a8319bddf39e0 (patch) | |
| tree | 9d8489582d8147bc7cca3fb611484b14653a750f /svx | |
| parent | 5ae845923499422540632456f1dafefa3ad49f4f (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
Change-Id: I4f1dcb3a25adeb0737e03aa9e9dfb193969250b6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103827
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Tested-by: Tomaž Vajngerl <quikee@gmail.com>
Diffstat (limited to 'svx')
| -rw-r--r-- | svx/source/table/tablecontroller.cxx | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/svx/source/table/tablecontroller.cxx b/svx/source/table/tablecontroller.cxx index b28522619bc6..f61ea7605567 100644 --- a/svx/source/table/tablecontroller.cxx +++ b/svx/source/table/tablecontroller.cxx @@ -1534,11 +1534,12 @@ 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()) |