{"action":"created","issue":{"url":"https://api.github.com/repos/behdad/fonttools/issues/267","labels_url":"https://api.github.com/repos/behdad/fonttools/issues/267/labels{/name}","comments_url":"https://api.github.com/repos/behdad/fonttools/issues/267/comments","events_url":"https://api.github.com/repos/behdad/fonttools/issues/267/events","html_url":"https://github.com/behdad/fonttools/pull/267","id":74639074,"number":267,"title":"[subset] add option to prune unused ulUnicodeRange bits","user":{"login":"anthrotype","id":6939968,"avatar_url":"https://avatars.githubusercontent.com/u/6939968?v=3","gravatar_id":"","url":"https://api.github.com/users/anthrotype","html_url":"https://github.com/anthrotype","followers_url":"https://api.github.com/users/anthrotype/followers","following_url":"https://api.github.com/users/anthrotype/following{/other_user}","gists_url":"https://api.github.com/users/anthrotype/gists{/gist_id}","starred_url":"https://api.github.com/users/anthrotype/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/anthrotype/subscriptions","organizations_url":"https://api.github.com/users/anthrotype/orgs","repos_url":"https://api.github.com/users/anthrotype/repos","events_url":"https://api.github.com/users/anthrotype/events{/privacy}","received_events_url":"https://api.github.com/users/anthrotype/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-05-09T10:09:05Z","updated_at":"2015-05-09T18:18:27Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/behdad/fonttools/pulls/267","html_url":"https://github.com/behdad/fonttools/pull/267","diff_url":"https://github.com/behdad/fonttools/pull/267.diff","patch_url":"https://github.com/behdad/fonttools/pull/267.patch"},"body":"this is the same as https://github.com/behdad/fonttools/pull/191\r\nall the merge, rebase or cherry-pick would fail, so I was forced to redo the changes manually from a new branch.\r\n\r\nFrom the previous thread, the consensus was that we would have this option always on in the subsetter, and maybe we could think about having it in ttLib later.\r\n\r\n@behdad in the other PR about OS/2 fsFirstCharIndex, you suggested to use isUnicode() method of cmap subtables. I wonder if I should use that here as well. Currently I'm only checking platformID 3 and platEncID 1 and 10, because those are the only ones which are mentioned in the relevant [paragraph](http://www.microsoft.com/typography/otspec/os2.htm#ur) of OS/2 specification."},"comment":{"url":"https://api.github.com/repos/behdad/fonttools/issues/comments/100526145","html_url":"https://github.com/behdad/fonttools/pull/267#issuecomment-100526145","issue_url":"https://api.github.com/repos/behdad/fonttools/issues/267","id":100526145,"user":{"login":"anthrotype","id":6939968,"avatar_url":"https://avatars.githubusercontent.com/u/6939968?v=3","gravatar_id":"","url":"https://api.github.com/users/anthrotype","html_url":"https://github.com/anthrotype","followers_url":"https://api.github.com/users/anthrotype/followers","following_url":"https://api.github.com/users/anthrotype/following{/other_user}","gists_url":"https://api.github.com/users/anthrotype/gists{/gist_id}","starred_url":"https://api.github.com/users/anthrotype/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/anthrotype/subscriptions","organizations_url":"https://api.github.com/users/anthrotype/orgs","repos_url":"https://api.github.com/users/anthrotype/repos","events_url":"https://api.github.com/users/anthrotype/events{/privacy}","received_events_url":"https://api.github.com/users/anthrotype/received_events","type":"User","site_admin":false},"created_at":"2015-05-09T18:18:27Z","updated_at":"2015-05-09T18:18:27Z","body":"Ok, I just fixed it and squashed into the previous commit."}}