rylev opened PR #8672 from rylev:defined-unknonw
to bytecodealliance:main
:
Fixes #8645
rylev requested wasmtime-core-reviewers for a review on PR #8672.
rylev requested fitzgen for a review on PR #8672.
alexcrichton submitted PR review:
Thanks! Could this be written with a recursive helper that operates on a
&ComponentType
(or similar) and aLinkerInstance
? That'll help remove some minor duplication here and additionally handle the fully general case of all component imports.Additionally could this return an error for unsupported imports? For example this can't fill in the import of a component or import of a module at this time so I think those should return an error rather than being silently ignored.
And finally, mind adding a test or two for this?
rylev updated PR #8672.
rylev updated PR #8672.
rylev requested alexcrichton for a review on PR #8672.
alexcrichton submitted PR review.
alexcrichton has enabled auto merge for PR #8672.
alexcrichton merged PR #8672.
Last updated: Dec 23 2024 at 12:05 UTC