Skip to content

Commit

Permalink
napi: create napi_env as a real structure
Browse files Browse the repository at this point in the history
1. We define struct napi_env__ to include the isolate, the last
exception, and the info about the last error.

2. We instantiate one struct napi_env__ during module registration and
we pass it into the FunctionCallbackInfo for all subsequent entries into
N-API when we create functions/accessors/finalizers.

Once module unloading will be supported we shall have to delete the
napi_env we create during module init.

There is a clear separation between public and private API wrt. env:

1. Public APIs assert that env is not nullptr as their first action.

2. Private APIs need not validate env. They assume it's not nullptr.

Fixes: nodejs#198
  • Loading branch information
Gabriel Schulhof committed Apr 4, 2017
1 parent 53828e8 commit 6985ed4
Show file tree
Hide file tree
Showing 4 changed files with 497 additions and 403 deletions.
Loading

0 comments on commit 6985ed4

Please sign in to comment.