-
Notifications
You must be signed in to change notification settings - Fork 158
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
Clarify object creation in named constructors #652
Conversation
This matches the text for normal constructors. I forgot to update this algorithm in 8e1b529.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hope at some point we can find a cleaner solution for "Perform the actions listed in the description".
Rethrow any exceptions. | ||
1. Let |O| be the result of [=converted to an ECMAScript value|converting=] |object| | ||
to an ECMAScript [=interface type=] value |I|. | ||
1. Assert: |O| is an object that implements |I|. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
link implements?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's no formal definition yet; I'm planning to add one soon, though.
index.bs
Outdated
1. Let |O| be the result of [=converted to an ECMAScript value|converting=] |object| | ||
to an ECMAScript [=interface type=] value |I|. | ||
1. Assert: |O| is an object that implements |I|. | ||
1. Assert: |O|.\[[Realm]] is equal to |realm|. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is equal to -> is?
index.bs
Outdated
1. Perform the actions listed in the description of |constructor| | ||
with |values| as the argument values | ||
and |object| as the <emu-val>this</emu-val> value. | ||
Rethrow any exceptions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is implied, no?
This matches the text for normal constructors. I forgot to update this
algorithm in 8e1b529.
Preview | Diff