Applying "Rotate Order" to the vertices different from the first crashes Synfig

[FirasH]

Synfig version & platform:
Master: 09220e

Issue description:

1. Launch Synfig
2. Select the "Spline Tool" (only Outline Layer)
3. Draw a Spline on canvas with 2 vertices (it is enough to cause the crash)
4. Go to the Parameters window and expand the list of "Vertices"
5. Select "Vertex 002" and right-click > Rotate Order

Synfig crashes consistently.

The issue can be reproduced even with more vertices. If you add more vertices in step 3, the crash is still reproducible if the vertex selected in step 5 is not the first one.

---

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

[rodolforg]

Duplication of #280 ?

[rodolforg]

Anyway, what does "rotate order" mean?
I can't figure it out.
And how is it supposed to work?

[ebarranco]

I remember trying to find out what it does some time ago and only found this on the wiki:

Rotate Order: Doesn't do anything.

Here in the "Specific actions for Width Points" section.

I always suspected that it was something to do with tangents back a the day when they had different colors and direction of Sewing Splines It was less ambiguous.

[morevnaproject]

Rotate Order changes order of vertices.
The effect of this operation is visible on Un-looped splines.

For example, right-clicking (1) on this vertex and choosing "Rotate Order" ...

...results in this:

[rodolforg]

Not reproducible in 1.3.15 (production. debug version has an assert that ruins everything :P)

BTW, @morevnaproject , this "rotate order" is a non-intuitive name. For me at least :P

[morevnaproject]

this "rotate order" is a non-intuitive name. For me at least :P

what could be the better name? ^__^

[FirasH]

Does not crash with current master 8b93aa1

[rodolforg]

@FirasH it crashes in debug build due to that pointed assert(). Asserts are ignored in production builds.

@morevnaproject I don't know XD What about "Set as first vertex"? :P

[morevnaproject]

"Set as first vertex" - this is better, but it doesn't tells that other points are reordered too. ^__^"

[FirasH]

Maybe something more specific like “Rotate order of vertices”

[morevnaproject]

That would be better than just "Rotate order". ^__^