Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JSTF: correct field name in JsftModList table #1165

Closed
PeterCon opened this issue Jul 6, 2024 · 2 comments
Closed

JSTF: correct field name in JsftModList table #1165

PeterCon opened this issue Jul 6, 2024 · 2 comments

Comments

@PeterCon
Copy link
Collaborator

PeterCon commented Jul 6, 2024

Description

The second field name in the JsftModList table currently is gsubLookupIndices, but the table is generic between GSUB and GPOS modifications, so should be named lookupIndices.

Page URL

https://learn.microsoft.com/en-us/typography/opentype/spec/jstf

Content source URL

https://github.com/MicrosoftDocs/typography/blob/live/typographydocs/opentype/spec/jstf.md

@PeterCon
Copy link
Collaborator Author

The current field name is an historical artefact related to a change introduced in Working Draft 5 (N0781, from March 2023) of OFF 5th edition.

In OT 1.9 and earlier, and in OFF up to WD4 for the 5th edn, there were two different JstfModList formats documented: JstfGSUBModList and JstfGPOSModList. The formats were identical except for names, and it made sense to combine them, though it's not clear to me where the request to make that change came from. (There wasn't an OT issue for it, and I haven't found discussion in the MPEG-OTSpec list.) In WD5, the separate formats were consolidated.

For OT 1.9.1, the changes that had been incorporated into the latest WD of OFF 5th Edn were incorporated into OT 1.9.1. When making this change, however, revising this field name was overlooked.

This is a technical error in OT 1.9.1 that shoudld be corrected as an erratum.

@PeterCon
Copy link
Collaborator Author

Erratum for OT 1.9.1 published.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant