Knot LPE ignores not-redundant 'z' segments

[su_v]

The 'Knot' path effects fails for crossings with not-redundant 'z' segments (straight lines closing a shape or path):
The results on canvas are not consistant and toggle between almost correct and (heavily) distorted paths:
- on load, only crossings with not-'z' segments are recognized and rendered
- toggling the visibility of the path effect results in a distorted path
- the same distorted path seems stored in the SVG source and used by other renderers (e.g. Firefox)

tested with Inkscape 0.48+devel r9623 on OS X 10.5.8
not reproduced with Inkscape 0.47 r22583

[su_v (suv-lp)]

test file drawn with Inkscape 0.47

[su_v (suv-lp)]

test file drawn with Inkscape 0.48+devel r9620

[jazzynico (jazzynico)]

Confirmed on Windows XP, Inkscape trunk revision 9674.

[Johan Engelen (johanengelen)]

Fixed in r9860.

Note:
LPE-Knot is horribly bugged when enabled and dragging knots around. I only found it usable after disabling it in the LPE dialog, drag path, then enable lpe-knot again...... :(

[su_v (suv-lp)]

Regression with closed curves in r9850 (other examples in prev. attached files work fine).

[su_v (suv-lp)]

oops, typo:

- Regression with closed curves in r9850
+ Regression with closed curves in r9860

[su_v (suv-lp)]

Another example of incorrect rendering of the 'Knot' lpe with closed curves (regression in r9860, tested against r9859):
closed curves now either disappear (comment #6) or appear cut off when rendered with the knot effect applied.

[Johan Engelen (johanengelen)]

Ok, fixed in r9862 then? :P

[su_v (suv-lp)]

r9862 looks fine (all examples above now render as expected).

thx, Johan :)

[su_v (suv-lp)]

inkscape trunk: r9862
inkscape 0.48.x: r9707