diff options
| author | Jason Ginchereau <jasongin@microsoft.com> | 2017-04-05 10:18:56 -0700 |
|---|---|---|
| committer | Michael Dawson <michael_dawson@ca.ibm.com> | 2017-04-07 09:40:21 -0400 |
| commit | 84602845c6f2a015b7b214751543d0793e45a5a9 (patch) | |
| tree | 66e9a1c46daf50d0910f6c22cca746107fa90516 /configure | |
| parent | 943d0853076437896963975d039e7786e5f74192 (diff) | |
| download | android-node-v8-84602845c6f2a015b7b214751543d0793e45a5a9.tar.gz android-node-v8-84602845c6f2a015b7b214751543d0793e45a5a9.tar.bz2 android-node-v8-84602845c6f2a015b7b214751543d0793e45a5a9.zip | |
n-api: Update property attrs enum to match JS spec
The napi_property_attributes enum used names and values from
v8::PropertyAttribute, but those negative flag names were outdated
along with the default behavior of a property being writable,
enumerable, and configurable unless otherwise specified. To match the
ES5 standard property descriptor those attributes should be positive
flags and should default to false unless otherwise specified.
PR-URL: https://github.com/nodejs/node/pull/12240
Fixes: https://github.com/nodejs/abi-stable-node/issues/221
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Diffstat (limited to 'configure')
0 files changed, 0 insertions, 0 deletions