Stream: git-wasmtime

Topic: wasmtime / PR #8791 wasmtime: Make Store context availabl...


view this post on Zulip Wasmtime GitHub notifications bot (Jun 13 2024 at 00:44):

jameysharp requested wasmtime-core-reviewers for a review on PR #8791.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 13 2024 at 00:44):

jameysharp requested elliottt for a review on PR #8791.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 13 2024 at 00:44):

jameysharp opened PR #8791 from jameysharp:call-hook-context to bytecodealliance:main:

Rather than digging only the embedder's store-specific data out and passing that along, pass access to the entire Store. This is the same thing we do for epoch interruption hooks, including the annoyance of having to take the callback out of the store temporarily while calling it to avoid having multiple mutable borrows.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 13 2024 at 01:03):

jameysharp commented on PR #8791:

I think the only other example of this pattern in Store is Store::limiter and its async variant. I don't want to change that today but I think it's worth discussing whether we should do that eventually.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 13 2024 at 14:55):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 13 2024 at 15:10):

alexcrichton merged PR #8791.


Last updated: Nov 22 2024 at 16:03 UTC