-
Notifications
You must be signed in to change notification settings - Fork 415
Move re-initialization logic into Rust/wasm-bindgen #947
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
Draft
logan-gatlin
wants to merge
24
commits into
main
Choose a base branch
from
logan/remove-reinit-for-unwind
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
0bd82a0
initial
logan-gatlin 192709b
added writeln
logan-gatlin eca25d3
fixed typo
logan-gatlin 4fa8e58
Added reinit hook
logan-gatlin 23b448b
Removed instance id
logan-gatlin 62f9763
comments
logan-gatlin f2af754
Fix versioning
logan-gatlin 8187101
Fix submodule and missing FinalizationRegistry
logan-gatlin f76d48a
Update wasm-bindgen version to use patch
logan-gatlin 11ef7cc
Fix doc tests
logan-gatlin d51763f
Update shim for new rust hook system
logan-gatlin dd60e74
Update submodule
logan-gatlin d12e18d
Refactored DO state to be contained entirely in Rust
logan-gatlin 9f1b4a1
Changed DO suffix
logan-gatlin 140c58d
Make examples use wasm-bindgen fork
logan-gatlin fe28a15
Fix clippy warning
logan-gatlin b97723e
fmt
logan-gatlin f912535
Fix ci
logan-gatlin 431f453
Fix CI
logan-gatlin 521233d
Fix version mismatch
logan-gatlin f38bc95
Fixes
logan-gatlin b86d087
tests
logan-gatlin 1c71557
Fixed leaking DOs
logan-gatlin ac104bc
Switch to other reinit hook
logan-gatlin 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
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
Submodule wasm-bindgen
updated
143 files
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,19 @@ | ||
| import { WorkerEntrypoint } from "cloudflare:workers"; | ||
| import * as exports from "./index.js"; | ||
|
|
||
| Error.stackTraceLimit = 100; | ||
|
|
||
| // Best-effort panic logging. Lost after wasm reinit since | ||
| // wasm-bindgen calls __wbg_reset_state internally and the | ||
| // shim has no hook to re-register. | ||
| if (exports.setPanicHook) { | ||
| exports.setPanicHook(function (message) { | ||
| console.error("Rust panic:", message); | ||
| }); | ||
| } | ||
|
|
||
| class Entrypoint extends WorkerEntrypoint {} | ||
|
|
||
| $HANDLERS | ||
|
|
||
| export default Entrypoint; |
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
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
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
Oops, something went wrong.
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.