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

Show file tree
Hide file tree
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
Next Next commit
Fix C++ lint issues
  • Loading branch information
@vmoroz
vmoroz committed Mar 4, 2022
commit 63532d7960ffda29080d0353817b98756770c55c
3 changes: 2 additions & 1 deletion src/js_native_api_v8.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -722,7 +722,8 @@ void Reference::SecondPassCallback(
return;
}
reference->_secondPassParameter = nullptr;
v8impl::RefTracker::FinalizeAll(&reference->_env->finalizing_queue, /*isEnvTeardown:*/false);
v8impl::RefTracker::FinalizeAll(
&reference->_env->finalizing_queue, /*isEnvTeardown:*/false);
}

} // end of namespace v8impl
Expand Down
3 changes: 2 additions & 1 deletion src/js_native_api_v8.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -364,7 +364,8 @@ class TryCatch : public v8::TryCatch {
if (HasCaught()) {
_env->last_exception.Reset(_env->isolate, Exception());
} else {
v8impl::RefTracker::FinalizeAll(&_env->finalizing_queue, /*isEnvTeardown:*/false);
v8impl::RefTracker::FinalizeAll(
&_env->finalizing_queue, /*isEnvTeardown:*/false);
}
}

Expand Down