diff options
| author | Noel Grandin <noel@peralex.com> | 2016-01-22 13:25:44 +0200 |
|---|---|---|
| committer | Thorsten Behrens <Thorsten.Behrens@CIB.de> | 2016-01-22 23:54:20 +0000 |
| commit | 3fc292f7b32f30b98dad208eb03e086b927d38a2 (patch) | |
| tree | 94c0fcc02274e24d354fda24d3bfb251a9c2e3a5 /basegfx/source/polygon/b2dsvgpolypolygon.cxx | |
| parent | d8df6631ac173a920be834096c522014732ac151 (diff) | |
loplugin:fpcomparison in basegfx
fix comparing of floating point values
Change-Id: I54db66968cb999514747171eed82082612e0cac8
Reviewed-on: https://gerrit.libreoffice.org/21708
Reviewed-by: Armin Le Grand <Armin.Le.Grand@cib.de>
Tested-by: Jenkins <ci@libreoffice.org>
Diffstat (limited to 'basegfx/source/polygon/b2dsvgpolypolygon.cxx')
| -rw-r--r-- | basegfx/source/polygon/b2dsvgpolypolygon.cxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/basegfx/source/polygon/b2dsvgpolypolygon.cxx b/basegfx/source/polygon/b2dsvgpolypolygon.cxx index 25773e7fac40..e472e97b6c61 100644 --- a/basegfx/source/polygon/b2dsvgpolypolygon.cxx +++ b/basegfx/source/polygon/b2dsvgpolypolygon.cxx @@ -471,10 +471,10 @@ namespace basegfx nY += nLastY; } - if( nX == nLastX && nY == nLastY ) + if( rtl::math::approxEqual(nX, nLastX) && rtl::math::approxEqual(nY, nLastY) ) continue; // start==end -> skip according to SVG spec - if( fRX == 0.0 || fRY == 0.0 ) + if( rtl::math::approxEqual(fRX, 0.0) || rtl::math::approxEqual(fRY, 0.0) ) { // straight line segment according to SVG spec aCurrPoly.append(B2DPoint(nX, nY)); @@ -857,8 +857,8 @@ namespace basegfx } else { - const bool bXEqual(aEdgeStart.getX() == aEdgeEnd.getX()); - const bool bYEqual(aEdgeStart.getY() == aEdgeEnd.getY()); + const bool bXEqual(rtl::math::approxEqual(aEdgeStart.getX(), aEdgeEnd.getX())); + const bool bYEqual(rtl::math::approxEqual(aEdgeStart.getY(), aEdgeEnd.getY())); if(bXEqual && bYEqual) { |