{"action":"created","issue":{"url":"https://api.github.com/repos/WebAssembly/design/issues/740","repository_url":"https://api.github.com/repos/WebAssembly/design","labels_url":"https://api.github.com/repos/WebAssembly/design/issues/740/labels{/name}","comments_url":"https://api.github.com/repos/WebAssembly/design/issues/740/comments","events_url":"https://api.github.com/repos/WebAssembly/design/issues/740/events","html_url":"https://github.com/WebAssembly/design/pull/740","id":168739528,"number":740,"title":"Use section codes instead of section names","user":{"login":"titzer","id":12184276,"avatar_url":"https://avatars.githubusercontent.com/u/12184276?v=3","gravatar_id":"","url":"https://api.github.com/users/titzer","html_url":"https://github.com/titzer","followers_url":"https://api.github.com/users/titzer/followers","following_url":"https://api.github.com/users/titzer/following{/other_user}","gists_url":"https://api.github.com/users/titzer/gists{/gist_id}","starred_url":"https://api.github.com/users/titzer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/titzer/subscriptions","organizations_url":"https://api.github.com/users/titzer/orgs","repos_url":"https://api.github.com/users/titzer/repos","events_url":"https://api.github.com/users/titzer/events{/privacy}","received_events_url":"https://api.github.com/users/titzer/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/WebAssembly/design/labels/binary%20format","name":"binary format","color":"5319e7"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/WebAssembly/design/milestones/2","html_url":"https://github.com/WebAssembly/design/milestones/MVP","labels_url":"https://api.github.com/repos/WebAssembly/design/milestones/2/labels","id":1138963,"number":2,"title":"MVP","description":"See [here](https://github.com/WebAssembly/design/blob/master/MVP.md) for details.","creator":{"login":"sunfishcode","id":4503403,"avatar_url":"https://avatars.githubusercontent.com/u/4503403?v=3","gravatar_id":"","url":"https://api.github.com/users/sunfishcode","html_url":"https://github.com/sunfishcode","followers_url":"https://api.github.com/users/sunfishcode/followers","following_url":"https://api.github.com/users/sunfishcode/following{/other_user}","gists_url":"https://api.github.com/users/sunfishcode/gists{/gist_id}","starred_url":"https://api.github.com/users/sunfishcode/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sunfishcode/subscriptions","organizations_url":"https://api.github.com/users/sunfishcode/orgs","repos_url":"https://api.github.com/users/sunfishcode/repos","events_url":"https://api.github.com/users/sunfishcode/events{/privacy}","received_events_url":"https://api.github.com/users/sunfishcode/received_events","type":"User","site_admin":false},"open_issues":31,"closed_issues":112,"state":"open","created_at":"2015-05-29T00:13:49Z","updated_at":"2016-08-23T06:03:37Z","due_on":null,"closed_at":null},"comments":25,"created_at":"2016-08-01T20:53:24Z","updated_at":"2016-08-25T10:49:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/WebAssembly/design/pulls/740","html_url":"https://github.com/WebAssembly/design/pull/740","diff_url":"https://github.com/WebAssembly/design/pull/740.diff","patch_url":"https://github.com/WebAssembly/design/pull/740.patch"},"body":"(rebasing onto 0xC instead of master)\r\n\r\nThis PR proposes uses section codes for known sections, which is more compact and easier to check in a decoder.\r\nIt allows for user-defined sections that have string names to be encoded in the same manner as before.\r\nThe scheme of using negative numbers proposed here also has the advantage of allowing a single decoder to accept the old (0xB) format and the new (0xC) format for the time being."},"comment":{"url":"https://api.github.com/repos/WebAssembly/design/issues/comments/242347540","html_url":"https://github.com/WebAssembly/design/pull/740#issuecomment-242347540","issue_url":"https://api.github.com/repos/WebAssembly/design/issues/740","id":242347540,"user":{"login":"titzer","id":12184276,"avatar_url":"https://avatars.githubusercontent.com/u/12184276?v=3","gravatar_id":"","url":"https://api.github.com/users/titzer","html_url":"https://github.com/titzer","followers_url":"https://api.github.com/users/titzer/followers","following_url":"https://api.github.com/users/titzer/following{/other_user}","gists_url":"https://api.github.com/users/titzer/gists{/gist_id}","starred_url":"https://api.github.com/users/titzer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/titzer/subscriptions","organizations_url":"https://api.github.com/users/titzer/orgs","repos_url":"https://api.github.com/users/titzer/repos","events_url":"https://api.github.com/users/titzer/events{/privacy}","received_events_url":"https://api.github.com/users/titzer/received_events","type":"User","site_admin":false},"created_at":"2016-08-25T10:49:43Z","updated_at":"2016-08-25T10:49:43Z","body":"On Wed, Aug 24, 2016 at 5:43 PM, KarlSchimpf <notifications@github.com>\nwrote:\n\n> I for one, do not like what is being proposed here. It adds a BIG wart for\n> decompression. The rules are very complex and make it much harder to\n> implement a \"general\" decompression tool.\n>\n> Since there are so few sections (when considering the size of a WASM file),\n> I'm surprised how much time is spent saving a few dozen bytes.\n>\n> I also don't like that some argue that we should presume all \"sections\" are\n> already known. This makes integrating \"decompression\" algorithms more\n> problematic, in that the decompression algorithms have no logical place to\n> be (optionally) added.\n>\n>\nI'm not sure I understand this comment, now that I've removed references to\nstrings for identifying unknown sections. The identification of sections is\njust a single byte, as opposed to a string before. How does that make\ndecompression more complicated?\n\n\n> Karl Schimpf\n>\n> Karl Schimpf\n>\n> On Wed, Aug 24, 2016 at 8:29 AM, titzer <notifications@github.com> wrote:\n>\n> > Also, technically a negative number could be extended with more all-1's\n> > upper bits, which would be another case to handle :-(\n> >\n> > —\n> > You are receiving this because you are subscribed to this thread.\n> > Reply to this email directly, view it on GitHub\n> > <https://github.com/WebAssembly/design/pull/740#issuecomment-242106774>,\n> > or mute the thread\n> > <https://github.com/notifications/unsubscribe-auth/\n> ALRbV9DPXf1PCy7j4jNXQBCAY12QEVRmks5qjGNegaJpZM4JZ_El>\n>\n> > .\n> >\n>\n> —\n> You are receiving this because you were mentioned.\n> Reply to this email directly, view it on GitHub\n> <https://github.com/WebAssembly/design/pull/740#issuecomment-242112158>,\n> or mute the thread\n> <https://github.com/notifications/unsubscribe-auth/ALnq1ICT_xeYd4NJPLFwfTrkTNRJ_l1Oks5qjGargaJpZM4JZ_El>\n> .\n>\n"}}
{"actor":{"display_login":"titzer"}}