basegfx/source/polygon/b2dsvgpolypolygon.cxx | 15 +--------------
1 file changed, 1 insertion(+), 14 deletions(-)
New commits:
commit 35b3abf5831fe05360f58dd7d292602785a55281
Author: Mike Kaganski <mike.kagan...@collabora.com>
AuthorDate: Mon Nov 4 21:06:15 2024 +0200
Commit: Mike Kaganski <mike.kagan...@collabora.com>
CommitDate: Tue Nov 5 05:35:12 2024 +0100
Simplify a bit
The expression can't be negative. It's a Pythagorean sum.
Change-Id: Ic07783c5291da71de6701a139227def2f1c4f77b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175989
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com>
diff --git a/basegfx/source/polygon/b2dsvgpolypolygon.cxx
b/basegfx/source/polygon/b2dsvgpolypolygon.cxx
index 7e525d5e1644..a7a25e6e17bb 100644
--- a/basegfx/source/polygon/b2dsvgpolypolygon.cxx
+++ b/basegfx/source/polygon/b2dsvgpolypolygon.cxx
@@ -543,20 +543,7 @@ namespace basegfx::utils
// of radicant solved for fRY,
// with s=fRX/fRY)
const double fRatio(fRX/fRY);
- const double fRadicant2(
- p1_prime.getY()*p1_prime.getY() +
-
p1_prime.getX()*p1_prime.getX()/(fRatio*fRatio));
- if( fRadicant2 < 0.0 )
- {
- // only trivial solution, one
- // of the axes 0 -> straight
- // line segment according to
- // SVG spec
- aCurrPoly.append(B2DPoint(nX, nY));
- continue;
- }
-
- fRY=sqrt(fRadicant2);
+ fRY=std::hypot(p1_prime.getY(),
p1_prime.getX()/fRatio);
fRX=fRatio*fRY;
// keep center_prime forced to (0,0)
