Stream: git-wasmtime

Topic: wasmtime / PR #10935 Remove `unsafe` from `Instance::from...


view this post on Zulip Wasmtime GitHub notifications bot (Jun 05 2025 at 20:24):

alexcrichton opened PR #10935 from alexcrichton:make-safe-function-safe to bytecodealliance:main:

I mistakenly thought that this would be an unsafe function due to reasons that aren't applicable, and I'm not aware of any reason why this function should be unsafe, so mark it safe.

<!--
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 (Jun 05 2025 at 20:24):

alexcrichton requested wasmtime-core-reviewers for a review on PR #10935.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 05 2025 at 20:24):

alexcrichton requested dicej for a review on PR #10935.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 05 2025 at 21:54):

dicej submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 05 2025 at 21:54):

dicej created PR review comment:

Is the sentence regarding safety in the doc comment above still relevant, or should it be removed?

view this post on Zulip Wasmtime GitHub notifications bot (Jun 05 2025 at 21:55):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 05 2025 at 21:55):

alexcrichton created PR review comment:

Oops no it should be removed

view this post on Zulip Wasmtime GitHub notifications bot (Jun 05 2025 at 21:55):

alexcrichton updated PR #10935.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 05 2025 at 21:56):

alexcrichton has enabled auto merge for PR #10935.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 05 2025 at 23:33):

alexcrichton updated PR #10935.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 05 2025 at 23:33):

alexcrichton has enabled auto merge for PR #10935.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 06 2025 at 16:45):

alexcrichton merged PR #10935.


Last updated: Dec 06 2025 at 07:03 UTC