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

Bug #1299461 reported by Jabiertxof
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Medium
Jabiertxof

Bug Description

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

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

Revision history for this message
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

Revision history for this message
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
Revision history for this message
su_v (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 (?):
<http://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/changes/11212>

Changed in inkscape:
importance: Undecided → Medium
milestone: none → 0.91
status: Confirmed → Triaged
tags: added: regression
Revision history for this message
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.

Revision history for this message
Johan Engelen (johanengelen) wrote :

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

Revision history for this message
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.

su_v (suv-lp)
summary: - Some LPE fails on apply to original, but cloned elements
+ Some LPE fails on apply to original, but cloned elements (rev >= 11212)
su_v (suv-lp)
Changed in inkscape:
milestone: 0.91 → 0.92
milestone: 0.92 → 0.91.1
su_v (suv-lp)
Changed in inkscape:
milestone: 0.91.1 → 0.92
Revision history for this message
Jabiertxof (jabiertxof) wrote :

fixed in master in 0aba9576

Changed in inkscape:
assignee: nobody → Jabiertxof (jabiertxof)
status: Triaged → Fix Committed
Revision history for this message
Jabiertxof (jabiertxof) wrote :

backported to 0.92.x on e12c9962

Revision history for this message
Qantas94Heavy (qantas94heavy) wrote :

Fix released with 0.92.4.

Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.