Stream: git-wasmtime

Topic: wasmtime / PR #14840 Add C API tests for component map va...

view this post on Zulip Wasmtime GitHub notifications bot (Oct 19 2026 at 19:18):

jrandolf opened PR #14840 from jrandolf:c-api-map-value-tests to bytecodealliance:main:

The C API supports map values, but only the map type was tested. This adds value round-trip tests for map<string, u32> and map<u32, list<string>>, including duplicate keys, and documents in val.h that map entries keep their order and may contain duplicate keys.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 19 2026 at 19:28):

jrandolf has marked PR #14840 as ready for review.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 19 2026 at 19:28):

jrandolf removed the review request for wasmtime-core-reviewers from PR #14840.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 19 2026 at 19:28):

jrandolf requested dicej for a review on PR #14840.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 19 2026 at 19:28):

jrandolf requested wasmtime-core-reviewers for a review on PR #14840.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 19 2026 at 21:44):

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

view this post on Zulip Wasmtime GitHub notifications bot (Oct 20 2026 at 16:36):

:thumbs_up: alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 20 2026 at 16:36):

:speech_balloon: alexcrichton created PR review comment:

Could these tests be folded together? I think it's fine to just have the more complicated one or just haev one.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 20 2026 at 16:36):

alexcrichton removed the review request for dicej from PR #14840.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 20 2026 at 18:58):

:memo: jrandolf submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 20 2026 at 18:58):

:speech_balloon: jrandolf created PR review comment:

Done

view this post on Zulip Wasmtime GitHub notifications bot (Oct 20 2026 at 18:58):

jrandolf edited PR #14840:

The C API supports map values, but only the map type was tested. This adds a value round-trip test for map<u32, list<string>>, including duplicate keys, and documents in val.h that map entries keep their order and may contain duplicate keys.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 20 2026 at 19:01):

jrandolf updated PR #14840.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 20 2026 at 19:57):

:thumbs_up: alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 20 2026 at 19:57):

alexcrichton added PR #14840 Add C API tests for component map values to the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 20 2026 at 20:23):

:check: alexcrichton merged PR #14840.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 20 2026 at 20:23):

alexcrichton removed PR #14840 Add C API tests for component map values from the merge queue.

Last updated: Oct 31 2026 at 23:03 UTC