cxxtest: curve-test.h fails
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Inkscape |
Triaged
|
Medium
|
Tomasz Boczkowski |
Bug Description
In lp:inkscape r12445, curve-test.h fails with "make check".
Please could someone who is more familiar with 2Geom take a look?
In CurveTest:
../../src/
../../src/
In CurveTest:
../../src/
..
In CurveTest:
../../src/
../../src/
..
In CurveTest:
../../src/
../../src/
.
In CurveTest:
../../src/
Changed in inkscape: | |
importance: | Undecided → Medium |
status: | New → Triaged |
Changed in inkscape: | |
milestone: | none → 1.0 |
Changed in inkscape: | |
assignee: | nobody → Tomasz Boczkowski (penginsbacon) |
Changed in inkscape: | |
milestone: | 1.0-old → 1.0 |
Summary of failing test:
CurveTest: :testGetSegment Count
fixed by attached path. One of test segments was commented out during merge. The patch fixes that.
CurveTest: :testNodesInPat h
fixed by attached path. According to comment in line 93 zero lenght closing segments should not increase the node count.
CurveTest: :testLastFirstS egment
I'm not sure if the test is correct. Should closing an empty path add a segment to it?
CurveTest: :testLastPoint
When path is closed, last point is != first point. I'll post a patch fixing Geom::Path behaviour.
CurveTest: :testPenultimat ePoint :penultimate_ point() is dead code. It is not used since revision 8855. I think this test and penultimate_point method can be deleted.
SPCurve: