Stream: git-wasmtime

Topic: wasmtime / PR #13638 Use `MaybeUninit.write` instead of `...

Wasmtime GitHub notifications bot (Jun 14 2026 at 18:57):

WeiTheShinobi opened PR #13638 from WeiTheShinobi:update_stable_api to bytecodealliance:main:

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

• If this work has been discussed elsewhere, please include a link to that
  conversation. If it was discussed in an issue, just mention "issue #...".

• Explain why this change is needed. If the details are in an issue already,
  this can be brief.

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
-->

Solve a TODO:
Since std::mem::MaybeUninit<T>.write() has been stable, use it instead of the wrapper initialize().

Wasmtime GitHub notifications bot (Jun 14 2026 at 18:57):

WeiTheShinobi requested pchickey for a review on PR #13638.

Wasmtime GitHub notifications bot (Jun 14 2026 at 18:57):

WeiTheShinobi requested wasmtime-core-reviewers for a review on PR #13638.

Wasmtime GitHub notifications bot (Jun 14 2026 at 18:57):

WeiTheShinobi edited PR #13638:

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

• If this work has been discussed elsewhere, please include a link to that
  conversation. If it was discussed in an issue, just mention "issue #...".

• Explain why this change is needed. If the details are in an issue already,
  this can be brief.

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
-->

/// TODO: Replace calls to this function with
/// https://doc.rust-lang.org/nightly/std/mem/union.MaybeUninit.html#method.write
/// once it is stable.

Solve a TODO:
Since std::mem::MaybeUninit<T>.write() has been stable, use it instead of the wrapper initialize().

Wasmtime GitHub notifications bot (Jun 14 2026 at 18:58):

WeiTheShinobi edited PR #13638:

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

• If this work has been discussed elsewhere, please include a link to that
  conversation. If it was discussed in an issue, just mention "issue #...".

• Explain why this change is needed. If the details are in an issue already,
  this can be brief.

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
-->

/// TODO: Replace calls to this function with
/// https://doc.rust-lang.org/nightly/std/mem/union.MaybeUninit.html#method.write
/// once it is stable.
pub(crate) fn initialize<T>(dst: &mut std::mem::MaybeUninit<T>, val: T) {

Solve a TODO:
Since std::mem::MaybeUninit<T>.write() has been stable, use it instead of the wrapper initialize().

Wasmtime GitHub notifications bot (Jun 14 2026 at 19:00):

WeiTheShinobi converted PR #13638 Use MaybeUninit.write instead of initialize<T> to a draft

Wasmtime GitHub notifications bot (Jun 14 2026 at 19:05):

WeiTheShinobi updated PR #13638.

Wasmtime GitHub notifications bot (Jun 14 2026 at 19:12):

WeiTheShinobi has marked PR #13638 as ready for review.

Wasmtime GitHub notifications bot (Jun 14 2026 at 21:04):

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

Wasmtime GitHub notifications bot (Jun 15 2026 at 14:28):

:thumbs_up: alexcrichton submitted PR review:

Thanks!

Wasmtime GitHub notifications bot (Jun 15 2026 at 14:28):

alexcrichton added PR #13638 Use MaybeUninit.write instead of initialize<T> to the merge queue

Wasmtime GitHub notifications bot (Jun 15 2026 at 14:53):

:check: alexcrichton merged PR #13638.

Wasmtime GitHub notifications bot (Jun 15 2026 at 14:53):

alexcrichton removed PR #13638 Use MaybeUninit.write instead of initialize<T> from the merge queue

Last updated: Jul 29 2026 at 05:03 UTC