Skip to content

node-api: run Node-API finalizers in GC second pass #42208

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
vmoroz wants to merge 9 commits into from
Closed

node-api: run Node-API finalizers in GC second pass #42208

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
bfc9dd0
Implement finalizer_queue to run finalizers earlier
vmoroz Mar 4, 2022
63532d7
Fix C++ lint issues
vmoroz Mar 4, 2022
0a5c446
node-api: implement finalizer_queue to run finalizers earlier
vmoroz Mar 4, 2022
832409b
Fix C++ lint issues
vmoroz Mar 4, 2022
aaf43e2
Merge branch 'FinalizerQueue' of https://github.com/vmoroz/node into …
vmoroz Mar 4, 2022
d004b75
add public node_api_call_finalizers and unit tests
vmoroz Mar 11, 2022
518482f
implement node_api_set_finalizer_error_handler
vmoroz Mar 18, 2022
623d5a6
fix lint-cpp and lint-js issues
vmoroz Mar 18, 2022
66b25ac
fix lint-md issues
vmoroz Mar 18, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
fix lint-md issues
  • Loading branch information
@vmoroz
vmoroz committed Mar 18, 2022
commit 66b25ac995a6563dafd864c3e6d94d13f73709fd
4 changes: 2 additions & 2 deletions doc/api/n-api.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5286,8 +5286,8 @@ required in order to enable correct disposal of the reference.
### `node_api_set_finalizer_error_handler`

<!-- YAML
added: v18.0.0
napiVersion: Experimental
added: REPLACEME
napiVersion: REPLACEME
-->

```c
Expand Down