-
Notifications
You must be signed in to change notification settings - Fork 12
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
Initialization section fix #11
Comments
@lorenzogentile404 re-implement initialization and finalization sections. The new implementation is not that different from the new one, so take a look to it especially wrt address collisions (e.g., if an address sends funds to itself). |
In the initialization section you have 2 cases:
You have to schedule the initialization section for transaction end. So that if there is a roll back you can insert the extra 2 account rows below. Reverted transactionsYou then have 2 extra account rows where you decrement the
|
In the finalization section you have 2 cases:
Note: in the spec |
You have to add |
TX_INIT
misc/CCSR
/misc/CCRS
TX_FINL
finalizationDomSubStamp
shorthandsThe text was updated successfully, but these errors were encountered: