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

Warn on post table version 3 #47

Closed
HinTak opened this issue Jan 21, 2017 · 8 comments
Closed

Warn on post table version 3 #47

HinTak opened this issue Jan 21, 2017 · 8 comments

Comments

@HinTak
Copy link

@HinTak HinTak commented Jan 21, 2017

@khaledhosny

This comment has been minimized.

Copy link

@khaledhosny khaledhosny commented Jan 21, 2017

And that warning would be based on what spec?

@HinTak

This comment has been minimized.

Copy link
Author

@HinTak HinTak commented Jan 22, 2017

@khaledhosny

This comment has been minimized.

Copy link

@khaledhosny khaledhosny commented Jan 22, 2017

So fonts that is fully spec-complaint would produce warnings because some program some where is buggy? It is much easier to reduce this to a single warning “This is a font file, some application somewhere will handle some aspect of it incorrectly. You have been warned”, and call it a day.

@davelab6

This comment has been minimized.

Copy link
Contributor

@davelab6 davelab6 commented Jan 22, 2017

HinTak added a commit to HinTak/Font-Validator that referenced this issue Jan 22, 2017
@HinTak

This comment has been minimized.

Copy link
Author

@HinTak HinTak commented Jan 22, 2017

Firmware upgrades for printers (some program) is not a sensible demand to put upon users. Telling users that their printers have buggy firmware isn't practical, nor is asking users to upgrade/change OSes (again, some program) to get a better or correct rendering.

I was hoping this is a simple issue for contributors to have a go. Anyway, it is quicker for me to just make the change, than to respond to trolling.

Since FontVal runs stand-alone and just produces a report (and don't really do any actually rejecting/etc like otc), I'd just like the report to be as detailed as possible. It is up to the person reading the report to decide which parts are relevant to him/her; in the extreme case, he/she should stop reading the report if he/she does not see any item of interest.

@HinTak HinTak closed this Jan 22, 2017
@khaledhosny

This comment has been minimized.

Copy link

@khaledhosny khaledhosny commented Jan 25, 2017

The printing stuff is red herring, it is just an outdated note in a decade old spec that Apple is not known for keeping it up to date.

The Apple issue that actually triggered this (http://typedrawers.com/discussion/1945/truetype-based-fonts-and-post-table-version-3-0) has nothing to do with printing. It is an undocumented autohinting feature that seems to depend on an unknown number of variables and its extend is not even clear. Jumping quickly to just trigger warning here s premature at the best and even harmful on the long run as excessive unnecessary warnings tend to cause people to ignore them altogether and actually useful warnings become a needle in a haystack.

Not to mention that checking for post table version is not even enough, as evidenced from the forum discussion above, having glyph names other than what Apple expects (which is not clear either) will trigger the hinting issue. You can even have version 1 post table with 0 glyph names or partial coverage of the glyphs in the font, which will pass the check promoted here but still potentially trigger the autohinting issue (and the mythical printing issue).

@moyogo

This comment has been minimized.

Copy link
Contributor

@moyogo moyogo commented Jan 25, 2017

Good point Khaled, a post table version 1 can cause the bug just as well.

@HinTak

This comment has been minimized.

Copy link
Author

@HinTak HinTak commented Jan 25, 2017

I don't think so. The other post table version explicitly encodes the glyph names. Post table version 3 does not. hence the issue with non-conformant/unexpected glyph names.

Anyway, I am fine with adding more warnings on post table version 1 too, if you can identify such issues.

As I have repeated said a few times... I fully intend people to skip over and ignore part of the reports they don't like or think irrelevant to their usage. The warning message clearly says "Apple said...", so if you intend your fonts to work on microsoft platforms only or whatever, you are free to ignore it. Not sure why this is even worth going over and over about. The reports are meant to be as detailed as possible, as far as I am concerned. The reader decides what they want to do with that information.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.