From c7d202a61f9ce81b76b29e61252c23aa66daff07 Mon Sep 17 00:00:00 2001 From: Caolán McNamara Date: Sat, 1 Jul 2023 20:28:54 +0100 Subject: cid#1532376 Big parameter passed by value MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Change-Id: I6e0717ce5b035c1a4f4ec3abbd23ae60c98c8181 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/153851 Tested-by: Jenkins Reviewed-by: Caolán McNamara --- svx/source/table/tablecontroller.cxx | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) (limited to 'svx') diff --git a/svx/source/table/tablecontroller.cxx b/svx/source/table/tablecontroller.cxx index a5469e949ba5..af11e5c875f7 100644 --- a/svx/source/table/tablecontroller.cxx +++ b/svx/source/table/tablecontroller.cxx @@ -935,14 +935,13 @@ void SvxTableController::onFormatTable(const SfxRequest& rReq) SfxItemSet aNewAttr(rModel.GetItemPool()); // merge drawing layer text distance items into SvxBoxItem used by the dialog - SvxBoxItem aBoxItem(TextDistancesToSvxBoxItem(aNewAttr)); - - SvxBoxInfoItem aBoxInfoItem( aNewAttr.Get( SDRATTR_TABLE_BORDER_INNER ) ); + auto xBoxItem(std::make_shared(TextDistancesToSvxBoxItem(aNewAttr))); + auto xBoxInfoItem(std::make_shared(aNewAttr.Get(SDRATTR_TABLE_BORDER_INNER))); MergeAttrFromSelectedCells(aNewAttr, false); - FillCommonBorderAttrFromSelectedCells( aBoxItem, aBoxInfoItem ); - aNewAttr.Put( aBoxItem ); - aNewAttr.Put( aBoxInfoItem ); + FillCommonBorderAttrFromSelectedCells(*xBoxItem, *xBoxInfoItem); + aNewAttr.Put(*xBoxItem); + aNewAttr.Put(*xBoxInfoItem); // Fill in shadow properties. const SfxItemSet& rTableItemSet = rTableObj.GetMergedItemSet(); @@ -963,7 +962,7 @@ void SvxTableController::onFormatTable(const SfxRequest& rReq) rModel, false) ); // Even Cancel Button is returning positive(101) value, - xDlg->StartExecuteAsync([xDlg, this, aBoxItem, aBoxInfoItem](int nResult){ + xDlg->StartExecuteAsync([xDlg, this, xBoxItem, xBoxInfoItem](int nResult){ if (nResult == RET_OK) { SfxItemSet aNewSet(*(xDlg->GetOutputItemSet())); @@ -975,14 +974,14 @@ void SvxTableController::onFormatTable(const SfxRequest& rReq) //unchanged state back to their input properties if (aNewSet.GetItemState(SDRATTR_TABLE_BORDER, false) != SfxItemState::SET) { - aNewSet.Put(aBoxItem); + aNewSet.Put(*xBoxItem); } if (aNewSet.GetItemState(SDRATTR_TABLE_BORDER_INNER, false) != SfxItemState::SET) { - aNewSet.Put(aBoxInfoItem); + aNewSet.Put(*xBoxInfoItem); } - SvxBoxItemToTextDistances(aBoxItem, aNewSet); + SvxBoxItemToTextDistances(*xBoxItem, aNewSet); if (checkTableObject() && mxTable.is()) { -- cgit