Contour Segment representation ?

[BlackFoundry]

Wouldn't it be nice / meaningful to have Segment object too for glyphs' contours ?

[madig]

Where would you use it? One could start with a Contour.segments property that simply partitions points into a list based on the type == None or not.

[anthrotype]

i'm ok with adding a property like the one suggested to make it easier to iterate over points grouped by segment, but I'd advise against wrapping the list of Point objects into a Segment wrapper class that does nothing except slowing things down

[madig]

One possible use-case for a Segment would actually be attaching a bounds method for it, although that may be (better?) solved in a different way.

Also, each segment should contain the last point of the previous segment to make it easier to call fontTools' bezierTools functions on a segment.

[anthrotype]

a method or property of contour would be better, i think.
for the bounds of an array of points, you just use calcBounds function from fontTools.misc.arrayTools directly

[anthrotype]

defcon Contour has getter for segments which returns a list of lists of Points, and a removeSegment method as well (which looks quite complicated).
I don't know if we really need this. One can always use a segment-oriented pen(e.g. RecordingPen) with the contour.draw() method to obtain something very similar to this. Or Glyph.getPen() can be used to create contours as sequence of segments (as opposed to the getPointPen() which works in points).

[madig]

Let's rely on pens if they are available. We already do for bounds.