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

Remove [[Enumerate]] and associated reflective capabilities #274

Closed
bterlson opened this issue Feb 5, 2016 · 3 comments
Closed

Remove [[Enumerate]] and associated reflective capabilities #274

bterlson opened this issue Feb 5, 2016 · 3 comments
Assignees

Comments

@bterlson
Copy link
Contributor

bterlson commented Feb 5, 2016

At the January TC39 meeting and subsequent conversations we got consensus on removal of the [[Enumerate]] MOP operation and the associated Reflect.enumerate and proxy trap enumerate. A PR implementing this consensus is here: tc39/ecma262#367.

@goyakin
Copy link

goyakin commented Feb 17, 2016

Test262 tests

@abchatra abchatra added this to the 1.2 milestone Mar 11, 2016
@abchatra
Copy link
Contributor

@Yongqu
This is failing following ESNext kangax test case as well

print((function()
{
var passed = true;
var proxy = new Proxy({}, {
  enumerate: function() {
    passed = false;
  }
});
for(var key in proxy); // Should not throw, nor execute the 'enumerate' method.
return passed;
})());

Expected: true
Actual:Exception.

@abchatra abchatra removed this from the 1.2 milestone Mar 29, 2016
@abchatra
Copy link
Contributor

Removing 1.2 milestone due to backward compatibility concern. Please fix this in Master.

leirocks added a commit to leirocks/ChakraCore that referenced this issue May 12, 2016
leirocks added a commit to leirocks/ChakraCore that referenced this issue May 16, 2016
leirocks added a commit to leirocks/ChakraCore that referenced this issue Jun 6, 2016
leirocks added a commit to leirocks/ChakraCore that referenced this issue Aug 25, 2016
chakrabot pushed a commit that referenced this issue Aug 26, 2016
…ctive capabilities

Merge pull request #976 from leirocks:proxyenumerator

#274 Remove [[Enumerate]] and associated reflective capabilities

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

No branches or pull requests

6 participants