{"action":"created","issue":{"url":"https://api.github.com/repos/mozilla/rhino/issues/227","labels_url":"https://api.github.com/repos/mozilla/rhino/issues/227/labels{/name}","comments_url":"https://api.github.com/repos/mozilla/rhino/issues/227/comments","events_url":"https://api.github.com/repos/mozilla/rhino/issues/227/events","html_url":"https://github.com/mozilla/rhino/pull/227","id":99572362,"number":227,"title":"Strict mode in runtime","user":{"login":"tuchida","id":201790,"avatar_url":"https://avatars.githubusercontent.com/u/201790?v=3","gravatar_id":"","url":"https://api.github.com/users/tuchida","html_url":"https://github.com/tuchida","followers_url":"https://api.github.com/users/tuchida/followers","following_url":"https://api.github.com/users/tuchida/following{/other_user}","gists_url":"https://api.github.com/users/tuchida/gists{/gist_id}","starred_url":"https://api.github.com/users/tuchida/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tuchida/subscriptions","organizations_url":"https://api.github.com/users/tuchida/orgs","repos_url":"https://api.github.com/users/tuchida/repos","events_url":"https://api.github.com/users/tuchida/events{/privacy}","received_events_url":"https://api.github.com/users/tuchida/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-08-07T03:45:45Z","updated_at":"2015-08-31T22:36:31Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mozilla/rhino/pulls/227","html_url":"https://github.com/mozilla/rhino/pull/227","diff_url":"https://github.com/mozilla/rhino/pull/227.diff","patch_url":"https://github.com/mozilla/rhino/pull/227.patch"},"body":"Rhino implemented strict mode in parsing.\r\n```js\r\n'use strict';\r\nvar eval = 1;\r\n```\r\n```\r\njs: \"test.js\", line 2: \"eval\" is not a valid identifier for this use in strict mode.\r\njs: var eval = 1;\r\njs: ........^\r\n```\r\nBut do not implement in runtime.\r\n```js\r\nfunction a() {\r\n  return arguments.caller;  // expected to throw exception, but not.\r\n}\r\n```"},"comment":{"url":"https://api.github.com/repos/mozilla/rhino/issues/comments/136519057","html_url":"https://github.com/mozilla/rhino/pull/227#issuecomment-136519057","issue_url":"https://api.github.com/repos/mozilla/rhino/issues/227","id":136519057,"user":{"login":"gbrail","id":521323,"avatar_url":"https://avatars.githubusercontent.com/u/521323?v=3","gravatar_id":"","url":"https://api.github.com/users/gbrail","html_url":"https://github.com/gbrail","followers_url":"https://api.github.com/users/gbrail/followers","following_url":"https://api.github.com/users/gbrail/following{/other_user}","gists_url":"https://api.github.com/users/gbrail/gists{/gist_id}","starred_url":"https://api.github.com/users/gbrail/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gbrail/subscriptions","organizations_url":"https://api.github.com/users/gbrail/orgs","repos_url":"https://api.github.com/users/gbrail/repos","events_url":"https://api.github.com/users/gbrail/events{/privacy}","received_events_url":"https://api.github.com/users/gbrail/received_events","type":"User","site_admin":false},"created_at":"2015-08-31T22:36:31Z","updated_at":"2015-08-31T22:36:31Z","body":"On the deprecation thing, I was thinking about the fact that this PR changes the format of compiled bytecode, so that bytecode compiled with this version won't work with older versions. But like I said, that has happened before.\r\n\r\nI don't think we should remove additional \"deprecated\" functions from ScriptRuntime, however -- there is so much code out there that uses Rhino in unusual ways, I would be afraid of breaking it."}}