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
Add support for SHA384 #2538
Add support for SHA384 #2538
Conversation
Pull Request Test Coverage Report for Build 8250071679Details
💛 - Coveralls |
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.
ACK b3a7c14
If you were to review/ack #2473 @TheBlueMatt we could get it in, @Kixunil is MIA. |
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.
The last commit looks good.
Can now be rebased, #2473 is in. |
This is another truncated variant of SHA-512, so there's not a lot to add here, but its occasionally useful, though often in non-Bitcoin contexts.
Rebased, sorry I didn't have a chance to help with #2473. |
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.
ACK 4e3bb73
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.
ACK 4e3bb73
Based on #2473 as we need support for 48-byte arrays <-> hex conversions.
Closes #2483