delta/node-new.git/test/node-api/test_threadsafe_function/binding.c, branch main

node-api: emit uncaught-exception on unhandled tsfn callbacks

2022-05-27T17:32:08+00:00

PR-URL: https://github.com/nodejs/node/pull/36510
Fixes: https://github.com/nodejs/node/issues/36402
Reviewed-By: Michael Dawson 
Reviewed-By: James M Snell

node-api: faster threadsafe_function

2021-05-05T01:39:28+00:00

Invoke threadsafe_function during the same tick and avoid marshalling
costs between threads and/or churning event loop if either:

1. There's a queued call already
2. `Push()` is called while the main thread was running
   threadsafe_function

PR-URL: https://github.com/nodejs/node/pull/38506
Reviewed-By: Anna Henningsen 
Reviewed-By: Rich Trott 
Reviewed-By: James M Snell

test: fix typo in comment in binding.c

2021-04-13T13:27:17+00:00

PR-URL: https://github.com/nodejs/node/pull/38220
Reviewed-By: Richard Lau 
Reviewed-By: Michaël Zasso 
Reviewed-By: Beth Griggs 
Reviewed-By: James M Snell 
Reviewed-By: Colin Ihrig 
Reviewed-By: Luigi Pinca 
Reviewed-By: Darshan Sen

test: rename n-api to node-api

2021-02-06T13:03:38+00:00

This renames the macros used in the tests from `NAPI_*` to
`NODE_API_*`.

PR-URL: https://github.com/nodejs/node/pull/37217
Reviewed-By: Michael Dawson 
Reviewed-By: Beth Griggs 
Reviewed-By: Rich Trott

Revert "n-api: detect deadlocks in thread-safe function"

2020-05-23T15:45:42+00:00

This reverts commit d26ca06c16f497ffa5ac4845a27922d5058a9318 because
it breaks running the tests in debug mode, as
`v8::Isolate::GetCurrent()` is not allowed if no `Isolate` is active
on the current thread.

Refs: https://github.com/nodejs/node/pull/33276
Refs: https://github.com/nodejs/node/pull/32860

PR-URL: https://github.com/nodejs/node/pull/33453
Reviewed-By: Daniel Bevenius 
Reviewed-By: Colin Ihrig 
Reviewed-By: James M Snell 
Reviewed-By: David Carlier 
Reviewed-By: Gireesh Punathil 
Reviewed-By: Gabriel Schulhof

n-api: detect deadlocks in thread-safe function

2020-04-19T17:07:00+00:00

We introduce status `napi_would_deadlock` to be used as a return status
by `napi_call_threadsafe_function` if the call is made with
`napi_tsfn_blocking` on the main thread and the queue is full.

Fixes: https://github.com/nodejs/node/issues/32615
Signed-off-by: Gabriel Schulhof 
PR-URL: https://github.com/nodejs/node/pull/32860
Reviewed-By: Ben Noordhuis 
Reviewed-By: Michael Dawson 
Reviewed-By: Zeyu Yang

Revert "n-api: detect deadlocks in thread-safe function"

2020-04-16T19:52:12+00:00

This reverts commit aeb7084fe6446350ec032e9819746126811bf44f.

The solution creates incorrect behaviour on Windows.

Re: https://github.com/nodejs/node-addon-api/pull/697#issuecomment-612993476
Signed-off-by: Gabriel Schulhof 
PR-URL: https://github.com/nodejs/node/pull/32880
Reviewed-By: Ben Noordhuis 
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig 
Reviewed-By: Gireesh Punathil 
Reviewed-By: Chengzhong Wu 
Reviewed-By: Gerhard Stöbich

n-api: detect deadlocks in thread-safe function

2020-04-09T09:41:28+00:00

We introduce status `napi_would_deadlock` to be used as a return status
by `napi_call_threadsafe_function` if the call is made with
`napi_tsfn_blocking` on the main thread and the queue is full.

PR-URL: https://github.com/nodejs/node/pull/32689
Fixes: https://github.com/nodejs/node/issues/32615
Signed-off-by: Gabriel Schulhof 
Reviewed-By: James M Snell 
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig 
Reviewed-By: David Carlier 
Reviewed-By: Chengzhong Wu 
Reviewed-By: Michael Dawson

n-api: make func argument of napi_create_threadsafe_function optional

2019-06-22T02:23:22+00:00

PR-URL: https://github.com/nodejs/node/pull/27791
Refs: https://github.com/nodejs/node/issues/27592
Reviewed-By: Gabriel Schulhof

n-api: mark thread-safe function as stable

2019-01-22T06:30:11+00:00

Fixes: https://github.com/nodejs/node/issues/24249
PR-URL: https://github.com/nodejs/node/pull/25556
Reviewed-By: James M Snell 
Reviewed-By: Colin Ihrig 
Reviewed-By: Michael Dawson