Stream: git-wasmtime

Topic: wasmtime / PR #12849 Use explicit boxing for `InstanceAll...

view this post on Zulip Wasmtime GitHub notifications bot (Mar 26 2026 at 14:33):

cfallin submitted PR review:

LGTM; would it make sense to have an API method to wrap up Box::into_pin(try_new(...)) (something like Box::try_new_pin)?

view this post on Zulip Wasmtime GitHub notifications bot (Mar 26 2026 at 18:02):

github-actions[bot] added the label wasmtime:api on PR #12849.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 26 2026 at 19:09):

fitzgen updated PR #12849.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 26 2026 at 19:10):

fitzgen has enabled auto merge for PR #12849.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 26 2026 at 19:10):

fitzgen commented on PR #12849:

LGTM; would it make sense to have an API method to wrap up Box::into_pin(try_new(...)) (something like Box::try_new_pin)?

I don't feel super strongly, but I can do it in a follow up

view this post on Zulip Wasmtime GitHub notifications bot (Mar 26 2026 at 19:23):

fitzgen added PR #12849 Use explicit boxing for InstanceAllocator async methods to the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 26 2026 at 19:50):

fitzgen merged PR #12849.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 26 2026 at 19:50):

fitzgen removed PR #12849 Use explicit boxing for InstanceAllocator async methods from the merge queue.

Last updated: Apr 12 2026 at 23:10 UTC