scratchfoundation/paper.js
1
0
Fork 0
mirror of https://github.com/scratchfoundation/paper.js.git synced 2025-01-04 03:45:58 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

Clean-up @hkrish's changes a bit.

Browse source
This commit is contained in:
Jürg Lehni 2013-06-09 18:37:08 -07:00
parent f8e5fae826
commit 7faf2a9e70
2 changed files with 18 additions and 22 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/path/Curve.js
View file

@ -864,7 +864,8 @@ statics: {
step /= 2;
}
var pt = Curve.evaluate(values, minT, true, 0);
return new CurveLocation(this, minT, pt, null, point.getDistance(pt));
return new CurveLocation(this, minT, pt, null, null,
point.getDistance(pt));
},
getNearestPoint: function(point) {
@ -1024,13 +1025,13 @@ new function() { // Scope for methods that require numerical integration
}
};
}, new function() { // Scope for intersection using bezier fat-line clipping
function addLocation(locations, curve1, parameter, point, curve2, parameter2) {
function addLocation(locations, curve1, t1, point, curve2, t2) {
// Avoid duplicates when hitting segments (closed paths too)
var first = locations[0],
last = locations[locations.length - 1];
if ((!first || !point.equals(first._point))
&& (!last || !point.equals(last._point)))
locations.push(new CurveLocation(curve1, parameter, point, curve2, null, parameter2));
locations.push(new CurveLocation(curve1, t1, point, curve2, t2));
}
function addCurveIntersections(v1, v2, curve1, curve2, locations,
@ -1117,10 +1118,10 @@ new function() { // Scope for methods that require numerical integration
// (according to Numerical.TOLERANCE).
if (Math.abs(range1[1] - range1[0]) < /*#=*/ Numerical.TOLERANCE &&
Math.abs(range2[1] - range2[0]) < /*#=*/ Numerical.TOLERANCE) {
var t = (range1[0] + range1[1]) / 2;
var t2 = (range2[0] + range2[1]) / 2;
addLocation(locations, curve1, t,
Curve.evaluate(v1, t, true, 0), curve2, t2);
var t1 = (range1[0] + range1[1]) / 2,
t2 = (range2[0] + range2[1]) / 2;
addLocation(locations, curve1, t1,
Curve.evaluate(v1, t1, true, 0), curve2, t2);
break;
}
}

25
src/path/CurveLocation.js
View file

@ -37,7 +37,7 @@ var CurveLocation = Base.extend(/** @lends CurveLocation# */{
* @param {Point} point
*/
initialize: function CurveLocation(curve, parameter, point, _otherCurve,
_distance, _otherParameter) {
_otherParameter, _distance) {
// Define this CurveLocation's unique id.
this._id = CurveLocation._id = (CurveLocation._id || 0) + 1;
this._curve = curve;
@ -111,21 +111,16 @@ var CurveLocation = Base.extend(/** @lends CurveLocation# */{
getIntersection: function() {
var intersection = this._intersection;
if (!intersection && this._otherCurve) {
if( this._otherParameter ){
// if we have the parameter on the other curve use that for intersection
// rather than the point.
intersection = this._intersection = new CurveLocation(
this._otherCurve, this._otherParameter, null, this);
// Force calculate the other point from the parameter.
// DEBUG: @jlehni - Not sure why we have to do this? Shouldn't it auto-calculate
// upon first access?!
var param = this._otherParameter;
// If we have the parameter on the other curve use that for
// intersection rather than the point.
this._intersection = intersection = new CurveLocation(
this._otherCurve, param, param ? null : this._point, this);
// Force calculate the other point from the parameter.
// DEBUG: @jlehni - Not sure why we have to do this? Shouldn't
// it auto-calculate upon first access?!
if (param)
intersection.getPoint();
} else {
// _point is always defined for intersection
intersection = this._intersection = new CurveLocation(
this._otherCurve, null, this._point, this);
// Link both ways
}
intersection._intersection = this;
}
return intersection;