-----Original Message----- From: suv@...2204... [mailto:suv@...2204...] On Behalf Of ~suv Sent: 04 February 2011 16:39 To: Engelen, J.B.C. (Johan) Cc: inkscape-devel@lists.sourceforge.net; Krzysztof Kosiński Subject: Re: [Inkscape-devel] Updated 2geom -- what changed?
On 4/2/11 06:37, ~suv wrote:
On 2/2/11 22:26, J.B.C.Engelen@...1578... wrote:
I have not done extensive tests, Inkscape seems to build fine, runs
ok,
and what I tested (spiro LPE) works fine too. If there are any
troubles,
let me know.
tested with r10031 on OS X 10.5.8, default prefs, compared with
r10021:
select tool transforming with the mouse (stretch, scale, skew) seems broken
node tool node-editing of a path with auto-smooth nodes randomly (?)
converts
un-affected auto-smooth nodes into cusps.
In attached example:
- select e.g. an end node and move it: all auto-smooth nodes turn into cusp nodes
- now select all nodes (Ctrl+A) and move the selection of nodes (i.e. the whole path): auto-smooth nodes are restored
r10032 is nearly unusable - any transformations with the select tool (scale, rotate, stretch, skew) use the wrong reference point and the dragged corner gets completely detached from the position of the mouse pointer.
I temporarily fixed this. It is indeed a bug in 2geom: multiplication of matrices. Geom::Affine const affine(Geom::Translate(-norm) * rel_affine * Geom::Translate(norm) ); changed to Geom::Affine const affine( (Geom::Affine)Geom::Translate(-norm) * rel_affine * (Geom::Affine)Geom::Translate(norm) ); fixes it. Without the casts, the Translate matrices are effectively ignored. Perhaps with the new operator stuff, the compiler reorders the multiplication, Krzystof?
Cheers, Johan