delta/node-new.git/src/async_wrap.cc, branch pull-request-doc-updates

src: move public C++ APIs into src/api/*.cc

2019-01-31T23:43:27+00:00

This patch moves most of the public C++ APIs into src/api/*.cc
so that it's easier to tell that we need to be careful about
the compatibility of these code.

Some APIs, like `node::LoadEnvironmet()`, `node::Start()` and
`node::Init()` still stay in `node.cc` because they are still
very specific to our use cases and do not work quite well yet
for embedders anyway - we could not even manage to write cctest for
them at the moment.

PR-URL: https://github.com/nodejs/node/pull/25541
Reviewed-By: Gus Caplan

src: pass along errors from PromiseWrap instantiation

2019-01-29T19:01:58+00:00

PR-URL: https://github.com/nodejs/node/pull/25734
Reviewed-By: James M Snell 
Reviewed-By: Gus Caplan

src: reduce includes of node_internals.h

2019-01-21T06:54:42+00:00

PR-URL: https://github.com/nodejs/node/pull/25507
Reviewed-By: Daniel Bevenius 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Colin Ihrig 
Reviewed-By: Anna Henningsen 
Reviewed-By: James M Snell

src: schedule destroy hooks in BeforeExit early during bootstrap

2018-12-18T18:56:49+00:00

Instead of doing it in the `internalBinding('async_wrap')`
initialization whose first call is uncertain depending on how
the native modules are loaded in JS land during bootstrap.

PR-URL: https://github.com/nodejs/node/pull/25020
Reviewed-By: Jeremiah Senkpiel 
Reviewed-By: James M Snell 
Reviewed-By: Anna Henningsen

src: fix type mismatch warnings from missing priv

2018-12-04T03:44:15+00:00

Registration initialization functions are expected to have a 4th
argument, a void*, so add them where necessary to fix the warnings.

PR-URL: https://github.com/nodejs/node/pull/24737
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig

src: use custom TryCatch subclass

2018-12-03T16:25:06+00:00

PR-URL: https://github.com/nodejs/node/pull/24751
Reviewed-By: Joyee Cheung

src: elevate v8 namespaces referenced

2018-12-01T04:22:23+00:00

PR-URL: https://github.com/nodejs/node/pull/24657
Reviewed-By: Colin Ihrig 
Reviewed-By: Luigi Pinca 
Reviewed-By: Gireesh Punathil 
Reviewed-By: Minwoo Jung 
Reviewed-By: Franziska Hinkelmann

async_hooks: add HandleScopes to C++ embedder/addon API

2018-11-13T14:25:46+00:00

Add `HandleScope`s to the public C++ API for embedders/addons,
since these methods create V8 handles that should not leak into
the outer scopes.

In particular, for some of the methods it was not clear from
the function signatures that these functions previously
needed to be invoked with a `HandleScope`.

PR-URL: https://github.com/nodejs/node/pull/24285
Reviewed-By: Gus Caplan 
Reviewed-By: James M Snell 
Reviewed-By: Joyee Cheung 
Reviewed-By: Michael Dawson

src: bundle persistent-to-local methods as class

2018-11-13T04:27:40+00:00

Create a class `PersistentToLocal` which contains three methods,
`Strong`, `Weak`, and `Default`:

* `Strong` returns a `Local` from a strong persistent reference,
* `Weak` returns a `Local` from a weak persistent reference, and
* `Default` decides based on `IsWeak()` which of the above two to call.

These replace `node::StrongPersistentToLocal()`,
`node::WeakPersistentToLocal()`, and `node::PersistentToLocal()`,
respectively.

PR-URL: https://github.com/nodejs/node/pull/24276
Reviewed-By: Joyee Cheung 
Reviewed-By: Colin Ihrig 
Reviewed-By: Anna Henningsen 
Reviewed-By: James M Snell 
Reviewed-By: Refael Ackermann

src: fix Get() usage in async_wrap.cc

2018-11-07T17:29:22+00:00

PR-URL: https://github.com/nodejs/node/pull/24060
Reviewed-By: Daniel Bevenius 
Reviewed-By: Anna Henningsen 
Reviewed-By: Joyee Cheung