{"action":"created","issue":{"url":"https://api.github.com/repos/tc39/ecma262/issues/1137","repository_url":"https://api.github.com/repos/tc39/ecma262","labels_url":"https://api.github.com/repos/tc39/ecma262/issues/1137/labels{/name}","comments_url":"https://api.github.com/repos/tc39/ecma262/issues/1137/comments","events_url":"https://api.github.com/repos/tc39/ecma262/issues/1137/events","html_url":"https://github.com/tc39/ecma262/pull/1137","id":306502483,"node_id":"MDExOlB1bGxSZXF1ZXN0MTc1OTI4OTE5","number":1137,"title":"Editorial: clarify when early errors for functions apply","user":{"login":"bakkot","id":1653598,"node_id":"MDQ6VXNlcjE2NTM1OTg=","avatar_url":"https://avatars2.githubusercontent.com/u/1653598?v=4","gravatar_id":"","url":"https://api.github.com/users/bakkot","html_url":"https://github.com/bakkot","followers_url":"https://api.github.com/users/bakkot/followers","following_url":"https://api.github.com/users/bakkot/following{/other_user}","gists_url":"https://api.github.com/users/bakkot/gists{/gist_id}","starred_url":"https://api.github.com/users/bakkot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bakkot/subscriptions","organizations_url":"https://api.github.com/users/bakkot/orgs","repos_url":"https://api.github.com/users/bakkot/repos","events_url":"https://api.github.com/users/bakkot/events{/privacy}","received_events_url":"https://api.github.com/users/bakkot/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2018-03-19T15:06:54Z","updated_at":"2018-09-16T21:15:52Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/tc39/ecma262/pulls/1137","html_url":"https://github.com/tc39/ecma262/pull/1137","diff_url":"https://github.com/tc39/ecma262/pull/1137.diff","patch_url":"https://github.com/tc39/ecma262/pull/1137.patch"},"body":"Fixes #632.\r\n\r\nThis is _arguably_ normative, in that it changes the interpretation of a strict reading of the spec (which, I argue in the above issue, implies that `function f(a, a) { 'use strict'; }` would be legal, contrary to the intent of the author of the relevant text, implementations, and common sense).\r\n\r\nIn practice I think we're regarding this as either fixing a spec bug or merely a clarification, depending on whether you buy my above reading. So I've marked it as editorial.\r\n\r\nNote that I've also changed the rule for names to be based on the strictness of `BindingIdentifier` rather than the strictness of the entire `FunctionDeclaration` (or similar) production. I don't think that actually changes anything, since the full production can't be strict unless all of its parts are. I'm mainly making this change to minimize ambiguity and to better match #1091.\r\n\r\ncc @jugglinmike."},"comment":{"url":"https://api.github.com/repos/tc39/ecma262/issues/comments/421834992","html_url":"https://github.com/tc39/ecma262/pull/1137#issuecomment-421834992","issue_url":"https://api.github.com/repos/tc39/ecma262/issues/1137","id":421834992,"node_id":"MDEyOklzc3VlQ29tbWVudDQyMTgzNDk5Mg==","user":{"login":"jugglinmike","id":677252,"node_id":"MDQ6VXNlcjY3NzI1Mg==","avatar_url":"https://avatars2.githubusercontent.com/u/677252?v=4","gravatar_id":"","url":"https://api.github.com/users/jugglinmike","html_url":"https://github.com/jugglinmike","followers_url":"https://api.github.com/users/jugglinmike/followers","following_url":"https://api.github.com/users/jugglinmike/following{/other_user}","gists_url":"https://api.github.com/users/jugglinmike/gists{/gist_id}","starred_url":"https://api.github.com/users/jugglinmike/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jugglinmike/subscriptions","organizations_url":"https://api.github.com/users/jugglinmike/orgs","repos_url":"https://api.github.com/users/jugglinmike/repos","events_url":"https://api.github.com/users/jugglinmike/events{/privacy}","received_events_url":"https://api.github.com/users/jugglinmike/received_events","type":"User","site_admin":false},"created_at":"2018-09-16T21:15:52Z","updated_at":"2018-09-16T21:15:52Z","author_association":"CONTRIBUTOR","body":"Thanks for the recap, @jmdyck!\r\n\r\n@bterlson @bradleymeck @ljharb do you think we could move forward with this? I'll be happy to update gh-1158 once this is is available."}}
{"actor":{"display_login":"jugglinmike"}}