-
-
Notifications
You must be signed in to change notification settings - Fork 481
feat: add support for nogc types via BasicEnv
#1514
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.
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
d90111f
src: introduce `NogcEnv`; support for nogc finalizers
KevinEady 1ddf8f0
Address comments in 8 Jun 2024 Node-API meeting:
KevinEady 812d27c
sfinae FinalizeData::Wrapper
KevinEady 0ad6fcd
fix test/addon_build for experimental
KevinEady 64392b1
sfinae ObjectWrap::Finalize
KevinEady 18495a4
revert tests
KevinEady 70a8302
add finalizer_order test
KevinEady 26972bf
remove unnecessary defines
KevinEady 5d93fc3
add AddPostFinalizer test
KevinEady 94bbe58
renaming ideas
KevinEady c4e72eb
wip on docs for general finalization
KevinEady 4e8b367
switch to "basic" finalizer in docs
KevinEady de20ac4
Address review comments
KevinEady 6ee1ebf
finish renaming async finalizer to basic
KevinEady 13118f7
finish docs
KevinEady 1ad6eda
fix compilation error with duplicate `Finalizer` identifier
KevinEady ec9f521
attempt fix compilation issues #2
KevinEady a8f6380
fix docs
KevinEady 195ec28
Address review comments
KevinEady File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
switch to "basic" finalizer in docs
- Loading branch information
commit 4e8b367e134b8761d6427e04deeee8ce13d1ca4c
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.