Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#73 fp bindgen wasm #361

Closed
wants to merge 20 commits into from
Closed

#73 fp bindgen wasm #361

wants to merge 20 commits into from

Conversation

joepio
Copy link
Member

@joepio joepio commented Mar 29, 2022

#73

fp-bindgen is a tool for creating wasm plugin abstractions. It helps data and functions to pass the memory boundary, from host to plugin.

It's a really cool project. I've been playing with it for a bit and got it working.

Passing Store across the memory boundary

This is probably the hardest part of it all. I want to be able to do things like Store.query or query(store) from within a plugin. To do this, I need a Store available in the plugin. I want to prevent re-creating all the Store methods in the host (e.g. write a query and commit and get_resource function that the host makes available), which is how it is usually done in fp-bindgen examples.

One of the current limitations of fp-bindgen is sending host context to a runtime. When that is solved, I can start working on this issue.

One possible implementation is a new impl Storelike that is similar to Db. Or maybe we can move Db across the memory boundary without too much effort? It is cloned for every request, so it may not be that hard.

PR Checklist:

  • Link to related issue
  • Add changelog entry linking to issue
  • Added tests (if needed)
  • (If new feature) added in description / readme

@joepio joepio force-pushed the #73-fp-bindgen-wasm branch from af6c525 to 5ba9874 Compare April 3, 2022 07:55
@joepio joepio closed this Oct 26, 2022
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.

1 participant