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

Remove unnecessary check for other === start.

Browse source
This commit is contained in:
Jürg Lehni 2015-10-01 21:11:51 -05:00
parent de57a7fbc8
commit a808aaf0fa
1 changed files with 1 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -650,8 +650,6 @@ PathItem.inject(new function() {
// Intersections are always part of the resulting path, for // Intersections are always part of the resulting path, for
// all other segments check the winding contribution to see // all other segments check the winding contribution to see
// if they are to be kept. If not, the chain has to end here // if they are to be kept. If not, the chain has to end here
// TODO: We really should find a way to go backwards perhaps
// and try another path when this happens?
drawSegment(seg, null, 'discard', i, 'red'); drawSegment(seg, null, 'discard', i, 'red');
console.error('Excluded segment encountered, aborting #' console.error('Excluded segment encountered, aborting #'
+ pathCount + '.' + + pathCount + '.' +
@ -659,9 +657,7 @@ PathItem.inject(new function() {
break; break;
} }
var handleIn = path && seg._handleIn; var handleIn = path && seg._handleIn;
if (!path || !other || other === start) { if (!path || !other) {
// TODO: Is (other === start) check really required?
// Does that ever occur?
// Just add the first segment and all segments that have no // Just add the first segment and all segments that have no
// intersection. // intersection.
drawSegment(seg, null, 'add', i, 'black'); drawSegment(seg, null, 'add', i, 'black');