From a808aaf0fa0b4f7809b33f40544f8c31b1645f34 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=BCrg=20Lehni?= Date: Thu, 1 Oct 2015 21:11:51 -0500 Subject: [PATCH] Remove unnecessary check for other === start. --- src/path/PathItem.Boolean.js | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/path/PathItem.Boolean.js b/src/path/PathItem.Boolean.js index d8cdc6e9..9ba56009 100644 --- a/src/path/PathItem.Boolean.js +++ b/src/path/PathItem.Boolean.js @@ -650,8 +650,6 @@ PathItem.inject(new function() { // Intersections are always part of the resulting path, for // all other segments check the winding contribution to see // 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'); console.error('Excluded segment encountered, aborting #' + pathCount + '.' + @@ -659,9 +657,7 @@ PathItem.inject(new function() { break; } var handleIn = path && seg._handleIn; - if (!path || !other || other === start) { - // TODO: Is (other === start) check really required? - // Does that ever occur? + if (!path || !other) { // Just add the first segment and all segments that have no // intersection. drawSegment(seg, null, 'add', i, 'black');