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

Include call depth in vm.stopAndReturnStateDiff() results #6632

Closed
RPate97 opened this issue Dec 20, 2023 · 0 comments · Fixed by #7234
Closed

Include call depth in vm.stopAndReturnStateDiff() results #6632

RPate97 opened this issue Dec 20, 2023 · 0 comments · Fixed by #7234
Labels
T-feature Type: feature

Comments

@RPate97
Copy link
Contributor

RPate97 commented Dec 20, 2023

Component

Forge

Describe the feature you would like

The vm.stopAndReturnStateDiff() cheat code currently returns a flat array of AccountAccess structs with no information on the call depth. This makes it difficult to analyze the results if depth is important for your use case. It would be helpful to add a depth field to the AccountAccess struct to cover this.

Additional context

Since depth is already tracked internally, I would expect this to be relatively easy to implement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-feature Type: feature
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant