fonttools / fontbakery

🧁 A font quality assurance tool for everyone
https://fontbakery.readthedocs.io
Apache License 2.0
534 stars 99 forks source link

[com.google.fonts/check/outline_direction] Warn about paths without bounds #4745

Closed simoncozens closed 1 month ago

simoncozens commented 1 month ago

Description

Bodoni Moda's "OE" glyph looks like this:

    <TTGlyph name="OE" xMin="80" yMin="0" xMax="1850" yMax="1500">
      <contour>
        <pt x="784" y="1461" on="1"/>
      </contour>
      <contour>
        <pt x="311" y="750" on="1"/>
        <pt x="311" y="886" on="0"/>
        <pt x="356" y="1141" on="0"/>
        ...

Yep, that's a single-point contour. This currently crashes the outline direction check. But a single point contour is probably something we want to warn about anyway, so it seems a good idea to (a) make it not crash and (b) make it warn about the contour instead.

Checklist