-
Notifications
You must be signed in to change notification settings - Fork 253
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
Possible to override resolver behaviour? #39
Comments
would you mind giving a before and after code snippet for what motivated you to raise the issue? I bet you felt annoyed at a call site you'd love to have better. can you give an example of that? (ps you can also ignore my feedback, I'm just trying to learn how people are using this, so would appreciate it) |
Thanks @neilalexander. That kind of API sounds great, but I would like to better understand your usecase and API you have in your mind. Then we could come up with the concrete API before we merge in. |
I have a project which has a runtime where ideally we'd be able to control which functions and globals are available at runtime based on permissions/capabilities (which can change during execution). Right now we only have I don't have a specific API shape in mind here, but being able to pass in a callback which gets notified when resolving a function or global hasn't already been I'm not terribly familiar with this codebase but I'll be more than happy to work with you to help design an API shape :-) |
Hi @neilalexander, I just forgot to reply here.. we are planning to refactor the API in anyway #170, so we will revisit this issue after that. I will keep you posted! Thank you! |
Thanks so much for the update, I shall stay tuned! |
I think this is a relic as the API is wildly different since this was opened. Please open a new issue for any features you need! |
... so that instead of having to call
AddGlobal
,AddHostFunction
etc for everything that we want to inject, that we can instead set callbacks which return function pointers, allowing us to resolve missing functions, missing globals etc?The text was updated successfully, but these errors were encountered: