FiloSottile / FiloSottile/edwards25519
An academic question
- Dominant language
- Go
- Stars
- 188
- Forks
- 39
- PR merge metrics
- No merged PRs in 30d
Description
I've been reading this code while learning the field operations from the ground up. I'm also re-implementing the ideas in Rust on my own so I understand things better.
I noticed that in the field implementation, 'SetBytes' and 'Bytes' (bytes) do not appear to be symmetric. I thought it would be nice if the code makes their complementary nature more explicit.
I tried it out in Rust, and it reads better, I thought.
https://github.com/DeviPrasad/dev-tech-write/blob/main/techauth/tech/cryptography/code/c25519/src/field.rs#L431
Would it make sense if I try to write a Go version of the same? Would you like to comment?
Contributor guide
Assessment
This issue has not been assessed yet.