search

zcash / sapling-crypto

Zcash "Sapling" cryptography
Other
80 stars 66 forks source link

Improve test_uint32_rotr to exercise the by = by % 32 line. #99

Open defuse opened 5 years ago

defuse commented 5 years ago

If the first line of rotr was changed to let by = by % 33, for example, then the existing test wouldn't catch the bug.

ebfull commented 5 years ago

ACK