Stream: git-wasmtime

Topic: wasmtime / PR #12473 Add the `wasmtime_core::undo::Undo` ...


view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 19:13):

fitzgen opened PR #12473 from fitzgen:wasmtime-core-undo to bytecodealliance:main:

This is a helper for undoing partial/incremental side effects when their larger associated operation fails, so that state remains coherent and operations can have "all or nothing" semantics.

<!--
Please make sure you include the following information:

Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.html

Please ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 19:13):

fitzgen requested wasmtime-core-reviewers for a review on PR #12473.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 19:13):

fitzgen requested dicej for a review on PR #12473.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 19:32):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 19:32):

alexcrichton created PR review comment:

This'll mem::forget::<F> which I believe would leak, right?

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 19:32):

alexcrichton created PR review comment:

Perf-wise this might be worth the small amount of unsafe for storing these as ManuallyDrop<T> and ManuallyDrop<F> instead of Option?

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 19:32):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 19:36):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 19:36):

fitzgen created PR review comment:

Good catch

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 21:25):

fitzgen updated PR #12473.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 21:25):

fitzgen has enabled auto merge for PR #12473.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 21:37):

fitzgen added PR #12473 Add the wasmtime_core::undo::Undo helper to the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 22:00):

github-merge-queue[bot] removed PR #12473 Add the wasmtime_core::undo::Undo helper from the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 22:46):

fitzgen updated PR #12473.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 22:46):

fitzgen has enabled auto merge for PR #12473.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 23:00):

fitzgen added PR #12473 Add the wasmtime_core::undo::Undo helper to the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 23:27):

fitzgen merged PR #12473.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 29 2026 at 23:27):

fitzgen removed PR #12473 Add the wasmtime_core::undo::Undo helper from the merge queue.


Last updated: Feb 24 2026 at 04:36 UTC