{"action":"created","issue":{"url":"https://api.github.com/repos/adobe-type-tools/afdko/issues/400","repository_url":"https://api.github.com/repos/adobe-type-tools/afdko","labels_url":"https://api.github.com/repos/adobe-type-tools/afdko/issues/400/labels{/name}","comments_url":"https://api.github.com/repos/adobe-type-tools/afdko/issues/400/comments","events_url":"https://api.github.com/repos/adobe-type-tools/afdko/issues/400/events","html_url":"https://github.com/adobe-type-tools/afdko/issues/400","id":326628155,"node_id":"MDU6SXNzdWUzMjY2MjgxNTU=","number":400,"title":"[makeotfexe] another case of \"Unnecessary truncation of the Format 4 'cmap' subtable\"","user":{"login":"cjchapman","id":17747678,"node_id":"MDQ6VXNlcjE3NzQ3Njc4","avatar_url":"https://avatars3.githubusercontent.com/u/17747678?v=4","gravatar_id":"","url":"https://api.github.com/users/cjchapman","html_url":"https://github.com/cjchapman","followers_url":"https://api.github.com/users/cjchapman/followers","following_url":"https://api.github.com/users/cjchapman/following{/other_user}","gists_url":"https://api.github.com/users/cjchapman/gists{/gist_id}","starred_url":"https://api.github.com/users/cjchapman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cjchapman/subscriptions","organizations_url":"https://api.github.com/users/cjchapman/orgs","repos_url":"https://api.github.com/users/cjchapman/repos","events_url":"https://api.github.com/users/cjchapman/events{/privacy}","received_events_url":"https://api.github.com/users/cjchapman/received_events","type":"User","site_admin":false},"labels":[{"id":983125063,"node_id":"MDU6TGFiZWw5ODMxMjUwNjM=","url":"https://api.github.com/repos/adobe-type-tools/afdko/labels/makeotfexe","name":"makeotfexe","color":"5319e7","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2018-05-25T18:49:41Z","updated_at":"2018-07-31T16:12:22Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"@kenlunde and I found another scenario that may be related to: https://github.com/adobe-type-tools/afdko/issues/242\r\n\r\nSpecifically, when running this command (see the file `command.sh`):\r\n```\r\nmakeotf -f cidfont.ps -fi cidfontinfo -ff features -r -nS -omitMacNames -cs 3 -ch UniAKR-UTF32-H -ci Adobe-KR_sequences.txt\r\n```\r\nwe got the warning:\r\n```\r\nmakeotfexe [WARNING] <Format4TestAKR9-Regular> cmap{plat=3,script=1,lang=0}: cmap format 4 subtable was truncated to 2 segments, due to overflow with non-truncated version\r\n```\r\n~It's not clear if it really is impossible to fit the data into a cmap format 4 subtable, so the purpose of this issue is to remind us to investigate this further.~\r\n\r\nUPDATE: Peter Constable mentioned in an email to Ken and me (Chris):\r\n> I don't think the format 4 subtable has any limitation that would prevent encoding as many BMP characters as might be desired: there are 64K BMP characters, and format 4 can accommodate 64K codepoint range segments.\r\n\r\nHere's the full output:\r\n```\r\ncchapman-macOS:Downloads cchapman$ cd Format4TestAKR9\r\ncchapman-macOS:Format4TestAKR9 cchapman$ chmod +x command.sh \r\ncchapman-macOS:Format4TestAKR9 cchapman$ ./command.sh \r\nmakeotf [Warning] Could not find an Adobe CMAP Mac encoding mapping file. If you want a Mac cmap subtable, please specify the Mac CMAP encoding file with the '-cm' option.\r\nmakeotf [Note] setting the USE_TYPO_METRICS OS/2 fsSelection bit 7 from fontinfo keyword.\r\nmakeotf [Note] setting the WEIGHT_WIDTH_SLOPE_ONLY OS/2 fsSelection bit 8 from fontinfo keyword.\r\nmakeotf [Note] setting the OBLIQUE OS/2 fsSelection bit 9 from fontinfo keyword.\r\nmakeotfexe [WARNING] <Format4TestAKR9-Regular> cmap{plat=3,script=1,lang=0}: cmap format 4 subtable was truncated to 2 segments, due to overflow with non-truncated version\r\n makeotfexe [WARNING] <Format4TestAKR9-Regular> cmap{plat=1,script=3,lang=0}: empty encoding\r\n makeotfexe [WARNING] <Format4TestAKR9-Regular> Macintosh Adobe CMap not seen\r\n makeotfexe [NOTE] <Format4TestAKR9-Regular> cmap{plat=0,script=5,lang=0}: Number of default Unicode Variation Sequence values 288\r\n makeotfexe [NOTE] <Format4TestAKR9-Regular> cmap{plat=0,script=5,lang=0}: Number of non-default UVS values 18\r\n \r\n Built release mode font 'Format4TestAKR9-Regular.otf' Revision 1.000\r\nDone.\r\ncchapman-macOS:Format4TestAKR9 cchapman$ \r\n```"},"comment":{"url":"https://api.github.com/repos/adobe-type-tools/afdko/issues/comments/409278160","html_url":"https://github.com/adobe-type-tools/afdko/issues/400#issuecomment-409278160","issue_url":"https://api.github.com/repos/adobe-type-tools/afdko/issues/400","id":409278160,"node_id":"MDEyOklzc3VlQ29tbWVudDQwOTI3ODE2MA==","user":{"login":"cjchapman","id":17747678,"node_id":"MDQ6VXNlcjE3NzQ3Njc4","avatar_url":"https://avatars3.githubusercontent.com/u/17747678?v=4","gravatar_id":"","url":"https://api.github.com/users/cjchapman","html_url":"https://github.com/cjchapman","followers_url":"https://api.github.com/users/cjchapman/followers","following_url":"https://api.github.com/users/cjchapman/following{/other_user}","gists_url":"https://api.github.com/users/cjchapman/gists{/gist_id}","starred_url":"https://api.github.com/users/cjchapman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cjchapman/subscriptions","organizations_url":"https://api.github.com/users/cjchapman/orgs","repos_url":"https://api.github.com/users/cjchapman/repos","events_url":"https://api.github.com/users/cjchapman/events{/privacy}","received_events_url":"https://api.github.com/users/cjchapman/received_events","type":"User","site_admin":false},"created_at":"2018-07-31T16:12:22Z","updated_at":"2018-07-31T16:12:22Z","author_association":"CONTRIBUTOR","body":"Some additional info, just for future reference -- for the attached font, we get:\r\n4712 segments\r\n14721 glyphs that have to be explicitly listed at the end of the cmap subtable\r\n\r\nSo the size of the cmap format 4 subtable would be:\r\n16 bytes of header + 8 bytes/segment * 4712 segments +  2 bytes/glyph * 14721 glyphs = 67154 bytes\r\n"}}
{"actor":{"display_login":"cjchapman"}}