-
Notifications
You must be signed in to change notification settings - Fork 664
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
vms/platformvm
: Change AdvanceTimeTo
to modify passed-in parentState
#2489
Conversation
vms/platformvm
: Modify AdvanceTimeTo
to modify passed-in stateDiff
vms/platformvm
: Change AdvanceTimeTo
to modify passed-in stateDiff
vms/platformvm
: Change AdvanceTimeTo
to modify passed-in stateDiff
vms/platformvm
: Change AdvanceTimeTo
to modify passed-in parentState
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. I think given the introducing of the NewDiffOn
we can avoid the stateChanges
struct
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.
Just some nits. Also, worth considering renaming this file from state_changes.go
to advance_time.go
or something.
Why this should be merged
Whenever we consume
AdvanceTimeTo
, we always apply the changes immediately. Let's move that into this function directly.How this works
pretty straightforward
How this was tested
CI