Skip to content

Commit

Permalink
doc: cleanup n-api.md doc
Browse files Browse the repository at this point in the history
Partial doc cleanup as per
#20421

PR-URL: #20430
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Gabriel Schulhof <gabriel.schulhof@intel.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
mhdawson authored and MylesBorins committed Jul 10, 2018
1 parent eeb421c commit f4b9985
Showing 1 changed file with 8 additions and 6 deletions.
14 changes: 8 additions & 6 deletions doc/api/n-api.md
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ there to be one or more C++ wrapper modules that provide an inlineable C++
API. Binaries built with these wrapper modules will depend on the symbols
for the N-API C based functions exported by Node.js. These wrappers are not
part of N-API, nor will they be maintained as part of Node.js. One such
example is: [node-api](https://github.com/nodejs/node-api).
example is: [node-addon-api](https://github.com/nodejs/node-addon-api).

In order to use the N-API functions, include the file
[node_api.h](https://github.com/nodejs/node/blob/master/src/node_api.h)
Expand Down Expand Up @@ -88,7 +88,9 @@ typedef enum {
napi_generic_failure,
napi_pending_exception,
napi_cancelled,
napi_status_last
napi_escape_called_twice,
napi_handle_scope_mismatch,
napi_callback_scope_mismatch
} napi_status;
```
If additional information is required upon an API returning a failed status,
Expand Down Expand Up @@ -610,7 +612,7 @@ that has a loop which iterates through the elements in a large array:
```C
for (int i = 0; i < 1000000; i++) {
napi_value result;
napi_status status = napi_get_element(e, object, i, &result);
napi_status status = napi_get_element(env, object, i, &result);
if (status != napi_ok) {
break;
}
Expand Down Expand Up @@ -647,7 +649,7 @@ for (int i = 0; i < 1000000; i++) {
break;
}
napi_value result;
status = napi_get_element(e, object, i, &result);
status = napi_get_element(env, object, i, &result);
if (status != napi_ok) {
break;
}
Expand Down Expand Up @@ -2484,10 +2486,10 @@ performed using a N-API call).
property to be a JavaScript function represented by `method`. If this is
passed in, set `value`, `getter` and `setter` to `NULL` (since these members
won't be used).
- `data`: The callback data passed into `method`, `getter` and `setter` if
this function is invoked.
- `attributes`: The attributes associated with the particular property.
See [`napi_property_attributes`](#n_api_napi_property_attributes).
- `data`: The callback data passed into `method`, `getter` and `setter` if
this function is invoked.

### Functions
#### napi_get_property_names
Expand Down

0 comments on commit f4b9985

Please sign in to comment.