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

Improve getWinding() reliability by clamping curve-time to tMin <= t <= tMax

Browse source
This commit is contained in:
Jürg Lehni 2017-01-06 14:17:38 +01:00
parent 31aa68b2d1
commit 1c7d19bba0
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/path/PathItem.Boolean.js
View file

@ -637,7 +637,9 @@ PathItem.inject(new function() {
// the best quality. // the best quality.
var offsets = [0.48, 0.1, 0.9], var offsets = [0.48, 0.1, 0.9],
windingZero = { winding: 0, quality: 0 }, windingZero = { winding: 0, quality: 0 },
winding = windingZero; winding = windingZero,
tMin = /*#=*/Numerical.CURVETIME_EPSILON,
tMax = 1 - tMin;
for (var i = 0; i < offsets.length && winding.quality < 0.5; i++) { for (var i = 0; i < offsets.length && winding.quality < 0.5; i++) {
var length = totalLength * offsets[i]; var length = totalLength * offsets[i];
for (var j = 0, l = chain.length; j < l; j++) { for (var j = 0, l = chain.length; j < l; j++) {
@ -647,7 +649,7 @@ PathItem.inject(new function() {
var curve = entry.curve, var curve = entry.curve,
path = curve._path, path = curve._path,
parent = path._parent, parent = path._parent,
t = curve.getTimeAt(length), t = Numerical.clamp(curve.getTimeAt(length), tMin, tMax),
pt = curve.getPointAtTime(t), pt = curve.getPointAtTime(t),
// Determine the direction in which to check the winding // Determine the direction in which to check the winding
// from the point (horizontal or vertical), based on the // from the point (horizontal or vertical), based on the