mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bin unneeded variable

Browse Source 
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
This commit is contained in:
Tor Lillqvist
2016-01-19 11:58:03 +02:00
parent 8404dc3c76
commit 6dfcaa89b5
1 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
basegfx/source/polygon/b2dpolygoncutandtouch.cxx
View File

@@ -620,8 +620,6 @@ namespace basegfx
{ {
const B2DRange aRange(rCurr, rNext); const B2DRange aRange(rCurr, rNext);
const B2DVector aEdgeVector(rNext - rCurr); const B2DVector aEdgeVector(rNext - rCurr);
B2DVector aNormalizedEdgeVector(aEdgeVector);
aNormalizedEdgeVector.normalize();
bool bTestUsingX(fabs(aEdgeVector.getX()) > fabs(aEdgeVector.getY())); bool bTestUsingX(fabs(aEdgeVector.getX()) > fabs(aEdgeVector.getY()));
for(sal_uInt32 a(0L); a < nPointCount; a++) for(sal_uInt32 a(0L); a < nPointCount; a++)
@@ -634,7 +632,7 @@ namespace basegfx
{ {
const B2DVector aTestVector(aTestPoint - rCurr); const B2DVector aTestVector(aTestPoint - rCurr);
if(areParallel(aNormalizedEdgeVector, aTestVector)) if(areParallel(aEdgeVector, aTestVector))
{ {
const double fCut((bTestUsingX) const double fCut((bTestUsingX)
? aTestVector.getX() / aEdgeVector.getX() ? aTestVector.getX() / aEdgeVector.getX()