Some LPE fails on apply to original, but cloned elements (rev >= 11212)

Bug #1299461 reported by Jabiertxof on 2014-03-29
This bug affects 1 person
Affects Status Importance Assigned to Milestone

Bug Description

Steps to reproduce:
Create some path.
Select original path.
Apply envelope deformation.

I think it happens in all eefect than use doEffect_pwd2.
Tested with: concatenate_before_pwd2 = true;

Alvin Penner (apenner) wrote :

confirmed on Windows XP, Inkscape rev 13123

- draw path
- use Edit->Clone->Create Clone
- drag clone to right
- select original object
- save as clone_before.svg

Alvin Penner (apenner) wrote :

- add Path Effects->Envelope Deformation
- both objects collapse to very small size
- original path is at origin (0, 1052)
- clone is at top middle (322, 1040)
- both objects are less than a pixel in size
- save as clone_after.svg

- not reproduced on Inkscape 0.48.4

Changed in inkscape:
status: New → Confirmed
~suv (suv-lp) wrote :

Reproduced as described in comment #2 with r13229 on OS X 10.7.5

Based on tests with archived builds:
- not reproduced with rev <= 11208,
- reproduced with rev >= 11212;
the regression could be related to the changes in r11212 (?):

Changed in inkscape:
importance: Undecided → Medium
milestone: none → 0.91
status: Confirmed → Triaged
tags: added: regression
Johan Engelen (johanengelen) wrote :

It is caused by r11212, the change to sbasis.cpp on line 282.
However, this was part of a fix to 2geom because of some bug with paths where handles are at same location as nodes. (zero-length handle)... It has been quite a while since, and I no longer remember the details.

Johan Engelen (johanengelen) wrote :

Symptom: bounding box is wrong after lpe-enveloppe.cpp:line 232.

Johan Engelen (johanengelen) wrote :

The problem is that the clone wants to update too early, during construction of the patheffect setup. Originally this worked, because before the 2geom change, LPE Envelope threw an exception. After the fix, 2geom no longer throws and the LPE calculation returns some supersmall item, so the path (not clone) is updated *during* LPE set up.

~suv (suv-lp) on 2014-04-28
summary: - Some LPE fails on apply to original, but cloned elements
+ Some LPE fails on apply to original, but cloned elements (rev >= 11212)
~suv (suv-lp) on 2014-11-06
Changed in inkscape:
milestone: 0.91 → 0.92
milestone: 0.92 → 0.91.1
~suv (suv-lp) on 2015-09-30
Changed in inkscape:
milestone: 0.91.1 → 0.92
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers