-
Notifications
You must be signed in to change notification settings - Fork 184
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: nuking log_traits.nr
#8797
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
log_traits.nr
@@ -0,0 +1,25 @@ | |||
pub fn arr_to_be_bytes_arr<let L: u32>(fields: [Field; L]) -> [u8; L * 32] { |
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.
Moved these functions from log_traits.nr
+ refactored them such that they work with arrays instead of slices (we no longer need slices here with arithmetic over generics).
12826ad
to
1cb97db
Compare
fb9379f
to
3be7371
Compare
1cb97db
to
8d1132a
Compare
3be7371
to
1cd0dec
Compare
1cd0dec
to
94d31f3
Compare
399ab22
to
6911fc2
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.
This looks so satisfying 🤧
Fixes #8548