Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upIDL needs hook for subset of OrdinarySet #719
Comments
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment
Hide comment
Ms2ger
commented
Oct 25, 2016
|
So like #566? |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment
Hide comment
domenic
Oct 25, 2016
Member
Welp. Um, I guess I've reversed my position since #566, and think that we should indeed factor it out.
The alternative would be to add a giant warning in the ES source code like <!-- FILE AN ISSUE ON WEB IDL BEFORE CHANGING THIS IN ANY WAY -->, but I think a refactoring would probably be clearer.
|
Welp. Um, I guess I've reversed my position since #566, and think that we should indeed factor it out. The alternative would be to add a giant warning in the ES source code like |
added a commit
to TimothyGu/ecma262
that referenced
this issue
Sep 7, 2017
TimothyGu
referenced this issue
Sep 7, 2017
Merged
Editorial: Create OrdinarySetWithOwnDescriptor #992
bterlson
closed this
in
#992
Sep 7, 2017
added a commit
that referenced
this issue
Sep 7, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
annevk commentedOct 25, 2016
For heycam/webidl#201 we need a way to invoke step 3-onward of OrdinarySet.
Would something like OrdinarySetWithOwnDesc(O, P, V, Reciever, ownDesc) be okay? Should this new abstract operation be a subsection or sibling section of OrdinarySet?