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.

Sign up for GitHub

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

Jump to bottom

explicitly invoke [[HasOwnProperty]], [[GetProperyOf]], etc. more often #42

Closed
heycam opened this issue Feb 3, 2015 · 1 comment
Closed

explicitly invoke [[HasOwnProperty]], [[GetProperyOf]], etc. more often #42

heycam opened this issue Feb 3, 2015 · 1 comment
Labels
☕☕☕ difficulty:hard Excruciating ⌛⌛ duration:medium Shouldn't be too long to fix editorial Changes that do not affect how the standard is understood.

Comments

@heycam
Copy link
Collaborator

heycam commented Feb 3, 2015

There are many instances in the spec where it says something like "If O has an own property with name P, then ...". Given proxies, we really should be invoking [[HasOwnProperty]] here explicitly, because we'll want to be clear that the proxy handler is invoked. (And it's not really clear whether a proxy has "real" own properties anyway.)
@tobie tobie added ⌛⌛ duration:medium Shouldn't be too long to fix editorial Changes that do not affect how the standard is understood. ☕☕☕ difficulty:hard Excruciating labels Jun 19, 2016
@Ms2ger Ms2ger mentioned this issue Dec 18, 2018
Closed
@Ms2ger Ms2ger mentioned this issue Apr 16, 2019
Closed
@annevk
Copy link
Member

annevk commented Apr 30, 2020

As discussed in #601 this is not the behavior we want.

@annevk annevk closed this as completed Apr 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
☕☕☕ difficulty:hard Excruciating ⌛⌛ duration:medium Shouldn't be too long to fix editorial Changes that do not affect how the standard is understood.
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Editorial: Use some more abstract operations.
3 participants
@tobie @heycam @annevk