No need to check for overlap when determining value for unadjusted

Since setting only changes behavior if there actually is an overlap.
This commit is contained in:
Jürg Lehni 2015-10-05 05:52:35 +02:00
parent 05bc6afdbb
commit 90b4cf7292

View file

@ -577,10 +577,8 @@ PathItem.inject(new function() {
+ ', seg wi:' + seg._winding + ', seg wi:' + seg._winding
+ ', next wi:' + nextSeg._winding + ', next wi:' + nextSeg._winding
+ ', seg op:' + isValid(seg, true) + ', seg op:' + isValid(seg, true)
+ ', next op:' + (isValid(nextSeg, + ', next op:' + (isValid(nextSeg, !strict) || nextInter
!strict && inter._overlap) && isValid(nextInter._segment, !strict))
|| nextInter && isValid(nextInter._segment,
!strict && nextInter._overlap))
+ ', seg ov: ' + (seg._intersection + ', seg ov: ' + (seg._intersection
&& seg._intersection._overlap) && seg._intersection._overlap)
+ ', next ov: ' + (nextSeg._intersection + ', next ov: ' + (nextSeg._intersection
@ -609,9 +607,8 @@ PathItem.inject(new function() {
|| (!strict || isValid(seg, true)) || (!strict || isValid(seg, true))
// Even if next segment is not valid, its to which // Even if next segment is not valid, its to which
// we may switch might be, so count that too! // we may switch might be, so count that too!
&& (isValid(nextSeg, !strict && inter._overlap) && (isValid(nextSeg, !strict) || nextInter
|| nextInter && isValid(nextInter._segment, && isValid(nextInter._segment, !strict))
!strict && nextInter._overlap))
) )
? inter ? inter
// If it's no match, check the next linked intersection first, // If it's no match, check the next linked intersection first,