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

Deleting properties #94

Closed
jasongin opened this issue Feb 8, 2017 · 2 comments
Closed

Deleting properties #94

jasongin opened this issue Feb 8, 2017 · 2 comments

Comments

@jasongin
Copy link
Member

jasongin commented Feb 8, 2017

There should be NAPI functions for deleting named properties and indexed properties:

napi_status napi_delete_property(napi_env env, napi_value object, napi_propertyname name);
napi_status napi_delete_element(napi_env env, napi_value object, uint32_t i);

This is not super high-priority, since only one of the top native modules I scanned (couchbase) referenced v8::Object::Delete().
@jasongin jasongin mentioned this issue Jun 1, 2017
Closed
@mhdawson
Copy link
Member

nodejs/node#13924

@digitalinfinity
Copy link
Contributor

Addressed in nodejs/node#13934

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@digitalinfinity @mhdawson @jasongin