+
Skip to content

Conversation

punchcutter
Copy link
Collaborator

No description provided.

@punchcutter punchcutter requested a review from anthrotype March 30, 2020 18:21
@anthrotype
Copy link
Member

can you expand a little bit about the rationale for this change?

@punchcutter
Copy link
Collaborator Author

I can't see any reason why the BASE table would be ignored by default. I was subsetting a font with BASE table and with no explanation the BASE table was just dropped. Even with --passthrough-tables it still gets dropped because it's in the _drop_tables_default list. I can understand default dropping some of those other tables, but why BASE?

@anthrotype
Copy link
Member

ok yes, dropping by default may be too blunt.

Even with --passthrough-tables it still gets dropped

hm that's not good, indeed. Can we just fix that?

Ideally the subsetter would trim the BASE script coverage to match the subset. But I have to refresh my memory about BASE table to see how that would work.

@punchcutter
Copy link
Collaborator Author

Yes, I guess ideally it would trim to match the subset if necessary.

@anthrotype
Copy link
Member

I think here we should simply remove BASE from the _drop_tables_deafault list, so that it respects --no-subset-tables+=BASE or --passthrough-tables, but it is still dropped by default as subsetter does not know how to subset.

@anthrotype
Copy link
Member

Even though it's cumbersome, this should currently work to prevent subsetter to drop BASE table:

--drop-tables-=BASE --no-subset-tables+=BASE

Base automatically changed from master to main February 17, 2021 11:56
@behdad
Copy link
Member

behdad commented Aug 16, 2022

I'm happy to pass it through by default, since that does produce a correct font. @punchcutter can you rebase this plese?

@RickyDaMa
Copy link
Contributor

RickyDaMa commented Sep 26, 2025

This would be very helpful for some of the projects I'm helping to maintain. I'll take a look at rebasing this next week perhaps

@RickyDaMa
Copy link
Contributor

#3949 was merged so this PR can be closed now. Thanks @punchcutter for the original PR, my rebase retained your authorship 🚀

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.

4 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载