unified-font-object / ufo-spec

The official Unified Font Object specification source files.
http://unifiedfontobject.org
175 stars 30 forks source link

Why versionMajor is "integer" but versionMinor "non-negative integer"? #215

Closed anthrotype closed 1 year ago

anthrotype commented 1 year ago

Is there a reason why versionMajor is not defined as "non-negative" in the same way that versionMinor is?

image

I don't think a version can ever make any sense if it is negative, e.g. what does version -1.0 mean? Never seen one like that.

twardoch commented 1 year ago

"head.fontRevision" in the TT & OT specs is given as fixed type rather than ufixed. For this reason UFO speced this like so to be able to faithfully represent what might be in a font

justvanrossum commented 1 year ago

I'm not sure round-tripping a nonsensical value is a good enough reason. I for one would not be apposed to just change it to non-negative integer.