-
Notifications
You must be signed in to change notification settings - Fork 144
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
InitActor framework #76
Conversation
// TODO revisit on complete spec | ||
pub const METHOD_PLACEHOLDER: isize = 3; | ||
|
||
pub struct MethodParams(pub Vec<u8>); // TODO |
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.
why make the vec public?
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.
I did so because the vector will need to be pulled from the MethodParams when the functionality is implemented. It's possible in the future this can be removed in favour of implemented functions, but it will be a headache if this is kept private in the near future
@@ -11,6 +11,7 @@ use encoding::{blake2b_variable, Cbor, CodecProtocol, Error as EncodingError}; | |||
use leb128; | |||
use serde_cbor::Value::Bytes; | |||
use serde_cbor::{from_slice, to_vec}; | |||
use std::hash::Hash; |
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.
hot fire 🔥
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.
hash browns eh? @ansermino
@GregTheGreek @ansermino you guys good with this coming in? |
For now since the other types and implementations aren't built out, this can't be tested. Once it is able to be tested I will test functionality.
If anyone want's to review let me know and I won't make changes but I might tweak some things over the weekend and build on top of this if not, but will try to keep this PR as contained as possible.