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

Querier is passed to Migrate by reference #2007

Open
milapsheth opened this issue Oct 9, 2024 · 1 comment
Open

Querier is passed to Migrate by reference #2007

milapsheth opened this issue Oct 9, 2024 · 1 comment

Comments

@milapsheth
Copy link

A reference to the querier is passed to Migrate (&querier) as opposed to other wasmVM methods like Execute that receive querier as-is. Both Execute and Migrate expect the same querier type in their interface declaration.

Is there a reason for this discrepancy?

@chipshort
Copy link
Collaborator

I'm pretty sure there is no reason for this. Both passing by reference or by value work equally well in this case.

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

No branches or pull requests

2 participants