Undo recent change to Curve#getIntersection() since it breaks some boolean operations.

This commit is contained in:
Jürg Lehni 2013-05-02 11:32:24 -07:00
parent e8fed1bdf0
commit 840e879ba8

View file

@ -716,9 +716,7 @@ statics: {
// We need to provide the original left curve reference to the
// #getIntersections() calls as it is required to create the resulting
// CurveLocation objects.
getIntersections: function(v1, v2, curve1, curve2, locations,
// Pass on isFlat1 / isFlat2 parameters in iterative calls
isFlat1, isFlat2) {
getIntersections: function(v1, v2, curve1, curve2, locations) {
var bounds1 = this.getBounds(v1),
bounds2 = this.getBounds(v2);
/*#*/ if (options.debug) {
@ -738,13 +736,10 @@ statics: {
// because they have no control points at all, or are "flat enough"
// If the curve was flat in a previous iteration, we don't need to
// recalculate since it does not need further subdivision then.
if (!isFlat1)
isFlat1 = this.isLinear(v1)
|| this.isFlatEnough(v1, /*#=*/ Numerical.TOLERANCE);
if (!isFlat2)
isFlat2 = this.isLinear(v2)
|| this.isFlatEnough(v2, /*#=*/ Numerical.TOLERANCE);
if (isFlat1 && isFlat2) {
if ((this.isLinear(v1)
|| this.isFlatEnough(v1, /*#=*/ Numerical.TOLERANCE))
&& (this.isLinear(v2)
|| this.isFlatEnough(v2, /*#=*/ Numerical.TOLERANCE))) {
/*#*/ if (options.debug) {
new Path.Line({
from: [v1[0], v1[1]],
@ -778,12 +773,12 @@ statics: {
// Subdivide both curves, and see if they intersect.
// If one of the curves is flat already, no further subdivion
// is required.
var v1s = isFlat1 ? [v1] : this.subdivide(v1),
v2s = isFlat2 ? [v2] : this.subdivide(v2);
for (var i = 0, l = v1s.length; i < l; i++)
for (var j = 0, k = v2s.length; j < k; j++)
var v1s = this.subdivide(v1),
v2s = this.subdivide(v2);
for (var i = 0; i < 2; i++)
for (var j = 0; j < 2; j++)
this.getIntersections(v1s[i], v2s[j], curve1, curve2,
locations, isFlat1, isFlat2);
locations);
}
}
return locations;