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.

Sign up for GitHub

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

Jump to bottom

avm-simulator: nested calls from a static context should be static #6370

Closed
fcarreiro opened this issue May 13, 2024 · 0 comments · Fixed by #6414
Closed

avm-simulator: nested calls from a static context should be static #6370

fcarreiro opened this issue May 13, 2024 · 0 comments · Fixed by #6414
Assignees
@fcarreiro
Labels
C-avm Component: AVM related tickets (aka public VM)

Comments

@fcarreiro
Copy link
Contributor

Check public -> public and private -> public.
@fcarreiro fcarreiro added the C-avm Component: AVM related tickets (aka public VM) label May 13, 2024
@fcarreiro fcarreiro self-assigned this May 13, 2024
@github-project-automation github-project-automation bot moved this to Todo in A3 May 13, 2024
@fcarreiro fcarreiro mentioned this issue May 15, 2024
Merged
fcarreiro added a commit that referenced this issue May 15, 2024
@fcarreiro
fix(avm-simulator): nested calls should preserve static context (#6414)
44d7916 
Also change error messages to conform to the current ones. Makes all
`e2e_static_calls.test.ts` tests pass under the AVM.

Thanks @Thunkar for the heads up and fix.

Closes #6370.
@github-project-automation github-project-automation bot moved this from Todo to Done in A3 May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fcarreiro fcarreiro

Labels
C-avm Component: AVM related tickets (aka public VM)
Projects
A3
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(avm-simulator): nested calls should preserve static context AztecProtocol/aztec-packages
1 participant
@fcarreiro