-
Notifications
You must be signed in to change notification settings - Fork 62
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
Cleanup and Refactoring #72
Conversation
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
server/contracts/Core/RootChain.sol
Outdated
* Event for block submission logging | ||
* @noteice The event indicates the addition of a new Plasma block | ||
* @param blockNumber The index of the block in which a deposit transaction | ||
* is included |
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.
blockNumber: The number of the block that was submitted
server/contracts/Core/RootChain.sol
Outdated
@@ -185,6 +185,9 @@ contract RootChain is ERC721Receiver { | |||
smt = new SparseMerkleTree(); | |||
} | |||
|
|||
|
|||
/// @dev called by a Validator to append a Plamsa block to the Plasma chain |
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.
typo Plamsa
setChallenged is only called once and should be removed to make the contract shorter & easier to read.
- setChallenged exists as a separate function becuase of stack limit errors - changing getExitIndex function to return an impossible exit value in case a particular slot's exit is not found
…rc721 into cleanup-from-master
As #70 but without the messy rebase/fast forward