Skip to content

Component Support WIP - #346

Draft
martindevans wants to merge 3 commits into
bytecodealliance:mainfrom
martindevans:wasm-components-wip
Draft

martindevans wants to merge 3 commits into
bytecodealliance:mainfrom
martindevans:wasm-components-wip

Conversation

@martindevans

Copy link
Copy Markdown
Member

This is a rough first pass looking at what's required for wasm component support. I've pulled in everything from the component folder of the C-API (see here) and have wrapped some of the easiest bits in the normal boilerplate (creating handles etc).

I'm absolutely not intending to complete this work solo!

We can use this branch to gather up any component related bits until we have something that works and it worth merging into main.

Comment thread src/Components/ComponentValue.cs Outdated

@cyraid cyraid left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a reason for making these on to a single line vs multi-line? The multi-line is a bit more readable. :)

@UserMist

Copy link
Copy Markdown

I am currently making wasmtime bindings for Beef, language which is basically lower level C#, and I expect to mostly finish them in the next week or two. Once that's done, you can use my work as reference.

@phillipcaofph

Copy link
Copy Markdown
Contributor

Hi, just wanting to know if there is going to be any progress with this PR? I'm keen to have component support via this package if possible

@martindevans

Copy link
Copy Markdown
Member Author

I'm not personally planning to work on it anytime soon, but I will be very happy to merge PRs towards component support!

@phillipcaofph

Copy link
Copy Markdown
Contributor

@martindevans Shall I target PRs into your fork? Given there is progress there already

@martindevans

Copy link
Copy Markdown
Member Author

Yes please. Let's use this as our branch to develop component features, until we have something worth merging into master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants