Simple glyph flags capacity off by one?

[wezm]

I think I might have the reserved capacity for simple glyph flags off by one (too small) here:

allsorts/src/tables/glyf.rs

Line 391 in 1685cc2

let mut flags = Vec::with_capacity(number_of_contours);

[mikeday]

Why is it number_of_contours instead of number_of_coordinates since that's when the loop stops?

[wezm]

Right. I think it should be number_of_coordinates.