{"action":"created","issue":{"url":"https://api.github.com/repos/jscert/jsexplain/issues/24","repository_url":"https://api.github.com/repos/jscert/jsexplain","labels_url":"https://api.github.com/repos/jscert/jsexplain/issues/24/labels{/name}","comments_url":"https://api.github.com/repos/jscert/jsexplain/issues/24/comments","events_url":"https://api.github.com/repos/jscert/jsexplain/issues/24/events","html_url":"https://github.com/jscert/jsexplain/issues/24","id":342499102,"node_id":"MDU6SXNzdWUzNDI0OTkxMDI=","number":24,"title":"Psuedocode generation puts tuple bindings backwards","user":{"login":"IgnoredAmbience","id":98555,"node_id":"MDQ6VXNlcjk4NTU1","avatar_url":"https://avatars0.githubusercontent.com/u/98555?v=4","gravatar_id":"","url":"https://api.github.com/users/IgnoredAmbience","html_url":"https://github.com/IgnoredAmbience","followers_url":"https://api.github.com/users/IgnoredAmbience/followers","following_url":"https://api.github.com/users/IgnoredAmbience/following{/other_user}","gists_url":"https://api.github.com/users/IgnoredAmbience/gists{/gist_id}","starred_url":"https://api.github.com/users/IgnoredAmbience/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/IgnoredAmbience/subscriptions","organizations_url":"https://api.github.com/users/IgnoredAmbience/orgs","repos_url":"https://api.github.com/users/IgnoredAmbience/repos","events_url":"https://api.github.com/users/IgnoredAmbience/events{/privacy}","received_events_url":"https://api.github.com/users/IgnoredAmbience/received_events","type":"User","site_admin":false},"labels":[{"id":456275735,"node_id":"MDU6TGFiZWw0NTYyNzU3MzU=","url":"https://api.github.com/repos/jscert/jsexplain/labels/bug","name":"bug","color":"ee0701","default":true},{"id":954393670,"node_id":"MDU6TGFiZWw5NTQzOTM2NzA=","url":"https://api.github.com/repos/jscert/jsexplain/labels/generator","name":"generator","color":"5EBEFF","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-07-18T21:30:09Z","updated_at":"2018-07-19T08:05:08Z","closed_at":null,"author_association":"MEMBER","body":"On the proxies branch, tuple bindings are reversed from the actual order. This was 'fixed' in #16, but seemingly not for all outputs. The function that builds this data structure is a fold/map combo which was rather awkward to work with. Definitely needs review with a fresh pair of eyes.\r\n\r\nFor example, see the `init_object` function in JsInterpreter, although other examples likely exist too."},"comment":{"url":"https://api.github.com/repos/jscert/jsexplain/issues/comments/406191303","html_url":"https://github.com/jscert/jsexplain/issues/24#issuecomment-406191303","issue_url":"https://api.github.com/repos/jscert/jsexplain/issues/24","id":406191303,"node_id":"MDEyOklzc3VlQ29tbWVudDQwNjE5MTMwMw==","user":{"login":"brabalan","id":1021637,"node_id":"MDQ6VXNlcjEwMjE2Mzc=","avatar_url":"https://avatars3.githubusercontent.com/u/1021637?v=4","gravatar_id":"","url":"https://api.github.com/users/brabalan","html_url":"https://github.com/brabalan","followers_url":"https://api.github.com/users/brabalan/followers","following_url":"https://api.github.com/users/brabalan/following{/other_user}","gists_url":"https://api.github.com/users/brabalan/gists{/gist_id}","starred_url":"https://api.github.com/users/brabalan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brabalan/subscriptions","organizations_url":"https://api.github.com/users/brabalan/orgs","repos_url":"https://api.github.com/users/brabalan/repos","events_url":"https://api.github.com/users/brabalan/events{/privacy}","received_events_url":"https://api.github.com/users/brabalan/received_events","type":"User","site_admin":false},"created_at":"2018-07-19T08:05:08Z","updated_at":"2018-07-19T08:05:08Z","author_association":"MEMBER","body":"What is the problem in `init_object`? The only tuple binding I see is this:\r\n``` ocaml\r\n    let (pn, _term_) = p in\r\n```\r\ntranslated to\r\n``` js\r\n      var _term_ = p[1];var pn = p[0];\r\n```\r\n\r\nAlthough the order the binding is done is in reverse, the code seems to do the right thing."}}
{"actor":{"display_login":"brabalan"}}