Skip to content
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

Editorial: clarify that [[Construct]] implies [[Call]] #1187

Merged
merged 2 commits into from
May 14, 2018

Conversation

jmdyck
Copy link
Collaborator

@jmdyck jmdyck commented May 3, 2018

That is, an object with a [[Construct]] internal method must have a [[Call]] internal method.
Previously, this was assumed but not explicitly stated.

See https://stackoverflow.com/questions/50109554/

@bterlson
Copy link
Member

I think including "additionally", as in "additionally supports the construct method", would improve clarity here. I'll make that tweak and commit. Thanks @jmdyck.

jmdyck and others added 2 commits May 14, 2018 13:48
That is, an object with a [[Construct]] internal method
must have a [[Call]] internal method.
Previously, this was assumed but not explicitly stated.

See https://stackoverflow.com/questions/50109554/
@bterlson bterlson force-pushed the Construct-without-Call branch from a14d1d5 to 8bdc750 Compare May 14, 2018 21:15
@bterlson bterlson merged commit a956a03 into tc39:master May 14, 2018
ljharb pushed a commit to ljharb/ecma262 that referenced this pull request May 14, 2018
* Editorial: clarify that [[Construct]] implies [[Call]]

That is, an object with a [[Construct]] internal method
must have a [[Call]] internal method.
Previously, this was assumed but not explicitly stated.

See https://stackoverflow.com/questions/50109554/

* Editorial: Add 'additionally'
@jmdyck
Copy link
Collaborator Author

jmdyck commented May 15, 2018

The insertion of "additionally" causes the paragraph to have exactly the same problem that I was trying to solve: it said/says nothing about an object that supports [[Construct]] but not [[Call]].

My commit had the effect of saying that no such object exists.

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

Successfully merging this pull request may close these issues.

4 participants