fitzgen opened PR #12553 from fitzgen:serde-for-vec to bytecodealliance:main:
Depends on https://github.com/bytecodealliance/wasmtime/pull/12552
fitzgen requested wasmtime-core-reviewers for a review on PR #12553.
fitzgen requested wasmtime-default-reviewers for a review on PR #12553.
fitzgen requested cfallin for a review on PR #12553.
alexcrichton submitted PR review.
alexcrichton created PR review comment:
This'll probably work with our built-in bincode bits but it would hamper testing with JSON-like containers for example. Could this use
.push(elem)?and rely onsize_hintpurely for performance rather than correctness? I think it'd be good to continue to be general with collections vs form-fitting them too much lest they be too inflexible for future use(similar to the len/cap check below)
fitzgen submitted PR review.
fitzgen created PR review comment:
Ah yes, I forgot that the deserializer can be used with any encoded sequence, not just exactly that what the same type's serializer emitted. Will fix.
fitzgen updated PR #12553.
fitzgen has enabled auto merge for PR #12553.
fitzgen added PR #12553 Implement serialization and deserialization for our OOM-handling Vec to the merge queue
fitzgen merged PR #12553.
fitzgen removed PR #12553 Implement serialization and deserialization for our OOM-handling Vec from the merge queue
Last updated: Feb 24 2026 at 04:36 UTC