New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Normative: Remove arguments.caller poison pill #689

Merged
merged 1 commit into from Sep 29, 2016

Conversation

Projects
None yet
3 participants
@bterlson
Member

bterlson commented Sep 12, 2016

Implementations have not had this extension for many years so continuing
to poison pill it is needless. This removes all mention of arguments
object's "caller" property with the exception of a note that gives a bit
of history (mostly so people can still ctrl+f "caller").

Closes #670.

@littledan

This comment has been minimized.

Member

littledan commented Sep 12, 2016

Well-done spec change--great simplification of the arguments exotic object, happy to have the historical note, and appropriate prohibited extension. Good stuff all around.

Normative: Remove arguments.caller poison pill
Implementations have not had this extension for many years so continuing
to poison pill it is needless. This removes all mention of arguments
object's "caller" property with the exception of a note that gives a bit
of history (mostly so people can still ctrl+f "caller").

Closes #670.

leobalter added a commit to tc39/agendas that referenced this pull request Sep 12, 2016

bterlson added a commit to tc39/agendas that referenced this pull request Sep 12, 2016

@bterlson bterlson merged commit b9c327f into tc39:master Sep 29, 2016

bakkot added a commit to bakkot/ecma262 that referenced this pull request Oct 4, 2016

Editorial: Minor grammar correction regarding arguments.callee
This fixes two pluralization agreement issues introduced by tc39#689.

claudepache added a commit to claudepache/ecma262 that referenced this pull request Oct 18, 2016

Editiorial: Small corrections follwing tc39#689
* typo: ``"caller"` → `"caller"`
* bug: remove a leftover reference to a special [[HasProperty]]
implementation that is gone with tc39#689

bterlson added a commit that referenced this pull request Nov 1, 2016

Editiorial: Small corrections follwing #689 (#714)
* typo: ``"caller"` → `"caller"`
* bug: remove a leftover reference to a special [[HasProperty]]
implementation that is gone with #689

bterlson added a commit that referenced this pull request Nov 2, 2016

Editorial: Minor grammar correction regarding arguments.callee (#707)
This fixes two pluralization agreement issues introduced by #689.
@bakkot

This comment has been minimized.

Contributor

bakkot commented Nov 21, 2016

I PR'd a test262 update for this, but I thought we were going to stop merging normative changes until accompanying tests had gone in?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment