basegfx/source/polygon/b2dpolygoncutandtouch.cxx | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)
New commits: commit 6dfcaa89b542efac7561a956fb8bfaf39c3b89de Author: Tor Lillqvist <[email protected]> Date: Tue Jan 19 11:58:03 2016 +0200 Bin unneeded variable As far as I see, the only use of aNormalizedEdgeVector is as a parameter to areParallel(), and there is no need for that to be normalized, is there? Change-Id: I31ca09391894ab441e5b88dbdd636cf4b91ac5c2 diff --git a/basegfx/source/polygon/b2dpolygoncutandtouch.cxx b/basegfx/source/polygon/b2dpolygoncutandtouch.cxx index 71804b3..378c858 100644 --- a/basegfx/source/polygon/b2dpolygoncutandtouch.cxx +++ b/basegfx/source/polygon/b2dpolygoncutandtouch.cxx @@ -620,8 +620,6 @@ namespace basegfx { const B2DRange aRange(rCurr, rNext); const B2DVector aEdgeVector(rNext - rCurr); - B2DVector aNormalizedEdgeVector(aEdgeVector); - aNormalizedEdgeVector.normalize(); bool bTestUsingX(fabs(aEdgeVector.getX()) > fabs(aEdgeVector.getY())); for(sal_uInt32 a(0L); a < nPointCount; a++) @@ -634,7 +632,7 @@ namespace basegfx { const B2DVector aTestVector(aTestPoint - rCurr); - if(areParallel(aNormalizedEdgeVector, aTestVector)) + if(areParallel(aEdgeVector, aTestVector)) { const double fCut((bTestUsingX) ? aTestVector.getX() / aEdgeVector.getX() _______________________________________________ Libreoffice-commits mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
