Skip to content

Conversation

@rsheeter
Copy link
Contributor

@rsheeter rsheeter commented Mar 10, 2023

Upcoming changes, such as merging glyphs, need to be able to read and/or write things not readily expressed as a set of specific ids such as "all glyphs." To accomodate this update read access (previously done for write) controlled by a function rather than a fixed set of identifiers.

Hoisted out of #120 in hopes of turning that into a series of small more reviewable PRs.

@rsheeter rsheeter changed the base branch from main to clippy March 10, 2023 05:46
@rsheeter rsheeter force-pushed the orch branch 4 times, most recently from ceb5273 to 86b6ab6 Compare March 10, 2023 06:05
@rsheeter rsheeter marked this pull request as ready for review March 10, 2023 06:09
@rsheeter rsheeter changed the base branch from clippy to main March 10, 2023 06:13
@rsheeter rsheeter changed the title Make ACL system more flexible Make ACL system more flexible (static glyf, 1 of 3) Mar 10, 2023
Copy link
Member

@cmyr cmyr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants