diff options
author | Michael Weghorn <m.weghorn@posteo.de> | 2022-03-18 14:17:28 +0100 |
---|---|---|
committer | Michael Weghorn <m.weghorn@posteo.de> | 2022-03-18 15:28:44 +0100 |
commit | cfa5489982a84f847d86b8bf8ce49b25e033ed48 (patch) | |
tree | 23dbb5ece6bba799dccb05282bdb80875dd9470c /svx | |
parent | 8a4f58bcd76c2c5824b014efb35ed818907527b8 (diff) |
WeldEditView: Just pass new Size object as param
Just pass `aOutputSize` as parameter instead
of assigning that to `aSize` again and then
passing that one as parameter in
`WeldEditView::SetDrawingArea` and
overrides in derived classes.
`aSize.setHeight(aSize.Height())` can just be dropped
as an overly complicated way of keeping the
height as it is.
In `SidebarTextControl::SetDrawingArea`, `aOutputSize`
was already passed as param, so the new value of `aSize`
was ignored anyway.
Change-Id: I23192d3c5c85e4371a48774b3b8f854beb751b82
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/131741
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Diffstat (limited to 'svx')
-rw-r--r-- | svx/source/dialog/weldeditview.cxx | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/svx/source/dialog/weldeditview.cxx b/svx/source/dialog/weldeditview.cxx index d3f3c3f79c9e..42eb22c12223 100644 --- a/svx/source/dialog/weldeditview.cxx +++ b/svx/source/dialog/weldeditview.cxx @@ -1554,11 +1554,9 @@ void WeldEditView::SetDrawingArea(weld::DrawingArea* pDrawingArea) rDevice.SetBackground(aBgColor); Size aOutputSize(rDevice.PixelToLogic(aSize)); - aSize = aOutputSize; - aSize.setHeight(aSize.Height()); makeEditEngine(); - m_xEditEngine->SetPaperSize(aSize); + m_xEditEngine->SetPaperSize(aOutputSize); m_xEditEngine->SetRefDevice(&rDevice); m_xEditEngine->SetControlWord(m_xEditEngine->GetControlWord() | EEControlBits::MARKFIELDS); |