diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-09-13 10:20:59 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-09-13 15:46:47 +0200 |
| commit | 1b1afc10f4d90e37c5e81961b8b8902133e65b00 (patch) | |
| tree | fb6f68d743b334b8df4fc374afe68dc49fd6cbb8 /writerfilter | |
| parent | cc233992dfe0fa9732c24774852d037d161546ce (diff) | |
Fraction: make conversion operators and constructor explicit
and simplify some of the calculations that needed to be changed.
Which resulted in one unit test needing to change by one pixel,
let's hope not an indication of a real problem.
Change-Id: Ie56434f35f4e58d21ee6f671392e93dc7542fca3
Reviewed-on: https://gerrit.libreoffice.org/42240
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'writerfilter')
| -rw-r--r-- | writerfilter/source/dmapper/WrapPolygonHandler.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/writerfilter/source/dmapper/WrapPolygonHandler.cxx b/writerfilter/source/dmapper/WrapPolygonHandler.cxx index 5d41154e9a9f..010cf2e195a6 100644 --- a/writerfilter/source/dmapper/WrapPolygonHandler.cxx +++ b/writerfilter/source/dmapper/WrapPolygonHandler.cxx @@ -100,8 +100,8 @@ WrapPolygon::Pointer_t WrapPolygon::correctWordWrapPolygon(const awt::Size & rSr awt::Point aMovePoint(aMove.operator long(), 0); pResult = move(aMovePoint); - Fraction aScaleX(nWrap100Percent, Fraction(nWrap100Percent) + aMove); - Fraction aScaleY(nWrap100Percent, Fraction(nWrap100Percent) - aMove); + Fraction aScaleX = nWrap100Percent / (nWrap100Percent + aMove); + Fraction aScaleY = nWrap100Percent / (nWrap100Percent - aMove); pResult = pResult->scale(aScaleX, aScaleY); Fraction aScaleSrcX(rSrcSize.Width, nWrap100Percent); |