no need to call GetNumerator() / GetDenominator() for Fraction

simplify the calculations - why have a Fraction and then just split
it up for the calculation?

Change-Id: I81af95e1ee1633f77e088c0990f308b1c169a6a2
Reviewed-on: https://gerrit.libreoffice.org/42242
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

1 files changed, 4 insertions, 12 deletions

diff --git a/svx/source/svdraw/svdxcgv.cxx b/svx/source/svdraw/svdxcgv.cxx
index 0debc4ab8e31..38db94a5aa37 100644
--- a/svx/source/svdraw/svdxcgv.cxx
+++ b/svx/source/svdraw/svdxcgv.cxx
@@ -402,18 +402,10 @@ void SdrExchangeView::ImpPasteObject(SdrObject* pObj, SdrObjList& rLst, const Po
     MapUnit eDstMU=mpModel->GetScaleUnit();
     FrPair aMapFact(GetMapFactor(eSrcMU,eDstMU));
     Fraction aDstFr(mpModel->GetScaleFraction());
-    nSizX*=aMapFact.X().GetNumerator();
-    nSizX*=rMap.GetScaleX().GetNumerator();
-    nSizX*=aDstFr.GetDenominator();
-    nSizX/=aMapFact.X().GetDenominator();
-    nSizX/=rMap.GetScaleX().GetDenominator();
-    nSizX/=aDstFr.GetNumerator();
-    nSizY*=aMapFact.Y().GetNumerator();
-    nSizY*=rMap.GetScaleY().GetNumerator();
-    nSizX*=aDstFr.GetDenominator();
-    nSizY/=aMapFact.Y().GetDenominator();
-    nSizY/=rMap.GetScaleY().GetDenominator();
-    nSizY/=aDstFr.GetNumerator();
+    nSizX *= double(aMapFact.X() * rMap.GetScaleX() * aDstFr);
+    nSizX *= aDstFr.GetDenominator();
+    nSizY *= double(aMapFact.Y() * rMap.GetScaleY());
+    nSizY /= aDstFr.GetNumerator();
     long xs=nSizX;
     long ys=nSizY;
     Point aPos(rCenter.X()-xs/2,rCenter.Y()-ys/2);