SHA-512 Smaller Improvement Suggestions
- Dominant language
- C
- Stars
- 108
- Forks
- 95
- PR merge metrics
- No merged PRs in 30d
Description
### Description
During review of the PR that added a smaller SHA-512 option, a few improvements were suggested. The PR was merged without these improvements being made, but we should track implementing them to get a few more savings.
- https://github.com/ARMmbed/mbed-crypto/pull/178#discussion_r307462160
- https://github.com/ARMmbed/mbed-crypto/pull/178#discussion_r307477089
### Issue request type
[ ] Question
[X] Enhancement
[ ] Bug
Contributor guide
Research direction
Start by reading the two review discussions linked from PR #178 to identify the requested SHA-512 improvements; the issue does not name any files or tests. Because development has moved to Mbed TLS, locate the corresponding implementation there and confirm the agreed changes and size savings before proceeding.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- c
- Domain
- cryptography
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 15/100