-
Notifications
You must be signed in to change notification settings - Fork 3
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
Refactor rollbacks #197
Refactor rollbacks #197
Conversation
2bb6c2f
to
5edc248
Compare
5edc248
to
37e528a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, and great job, but I want do discuss one thing mostly:
- RefCell borrows should be done inside methods so we're 100% certain the ref is dropped,
and we don't panic.
.gitmodules
Outdated
@@ -1,3 +1,4 @@ | |||
[submodule "era-compiler-tester"] | |||
path = era-compiler-tester | |||
url = https://github.com/lambdaclass/era-compiler-tester.git | |||
branch = refactor-rollbacks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add EOL here
Now the VmState struct refers to the World and the VmState before got renamed for Execution.
This refactor abstract rollbacks to support an easier integration with new rollbackable fields. For that a new
World
struct is created to encapsulate all the those fields. See more #196. Link to the respective integrations prs: