{"action":"created","issue":{"url":"https://api.github.com/repos/fonttools/fonttools/issues/902","repository_url":"https://api.github.com/repos/fonttools/fonttools","labels_url":"https://api.github.com/repos/fonttools/fonttools/issues/902/labels{/name}","comments_url":"https://api.github.com/repos/fonttools/fonttools/issues/902/comments","events_url":"https://api.github.com/repos/fonttools/fonttools/issues/902/events","html_url":"https://github.com/fonttools/fonttools/issues/902","id":217862330,"number":902,"title":"maxp.maxSizeOfInstructions is not calculated?","user":{"login":"khaledhosny","id":93914,"avatar_url":"https://avatars3.githubusercontent.com/u/93914?v=3","gravatar_id":"","url":"https://api.github.com/users/khaledhosny","html_url":"https://github.com/khaledhosny","followers_url":"https://api.github.com/users/khaledhosny/followers","following_url":"https://api.github.com/users/khaledhosny/following{/other_user}","gists_url":"https://api.github.com/users/khaledhosny/gists{/gist_id}","starred_url":"https://api.github.com/users/khaledhosny/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/khaledhosny/subscriptions","organizations_url":"https://api.github.com/users/khaledhosny/orgs","repos_url":"https://api.github.com/users/khaledhosny/repos","events_url":"https://api.github.com/users/khaledhosny/events{/privacy}","received_events_url":"https://api.github.com/users/khaledhosny/received_events","type":"User","site_admin":false},"labels":[{"id":49202273,"url":"https://api.github.com/repos/fonttools/fonttools/labels/enhancement","name":"enhancement","color":"84b6eb","default":true}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-03-29T12:40:18Z","updated_at":"2017-04-03T11:22:28Z","closed_at":null,"body":"I was trying to make a font with defective `glyf` table glyphs for testing OTS, when I found that `Tests/ttx/data/TestTTF.ttf` is already rejected by OTS. This seems to be because `maxSizeOfInstructions` is set to 0, but some glyphs have `instructionLength` of 2.\r\n\r\nIt seems `maxSizeOfInstructions` is not calculated when generating `maxp` table, I’m not sure if this is intentional or not."},"comment":{"url":"https://api.github.com/repos/fonttools/fonttools/issues/comments/291115613","html_url":"https://github.com/fonttools/fonttools/issues/902#issuecomment-291115613","issue_url":"https://api.github.com/repos/fonttools/fonttools/issues/902","id":291115613,"user":{"login":"lemzwerg","id":1630245,"avatar_url":"https://avatars1.githubusercontent.com/u/1630245?v=3","gravatar_id":"","url":"https://api.github.com/users/lemzwerg","html_url":"https://github.com/lemzwerg","followers_url":"https://api.github.com/users/lemzwerg/followers","following_url":"https://api.github.com/users/lemzwerg/following{/other_user}","gists_url":"https://api.github.com/users/lemzwerg/gists{/gist_id}","starred_url":"https://api.github.com/users/lemzwerg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lemzwerg/subscriptions","organizations_url":"https://api.github.com/users/lemzwerg/orgs","repos_url":"https://api.github.com/users/lemzwerg/repos","events_url":"https://api.github.com/users/lemzwerg/events{/privacy}","received_events_url":"https://api.github.com/users/lemzwerg/received_events","type":"User","site_admin":false},"created_at":"2017-04-03T11:22:28Z","updated_at":"2017-04-03T11:22:28Z","body":"OK, just rereading Greg's e-mail...\r\n\r\nThe sentence 'Tools like VTT or ttfautohint [...] are apparently wrong' is highly debatable :-) Given that no implementation is actually using the max value, and that the (former) de-facto tool for creating bytecode (VTT)  disagrees or disagreed for a long time with Greg's interpretation, and that the wording in the specification is still ambiguous in the newest OpenType specification, we have to live with the dichotomy of two (or even more) valid values.  Using the maximum value of the bytecode sizes in the `fpgm`, `prep`, and `glyf` does certainly work.  Admittedly, it might be too large, but it doesn't do any harm."}}
{"actor":{"display_login":"lemzwerg"}}