Co-dfns / mystika

High-end Cryptographic Library
GNU Affero General Public License v3.0
43 stars 26 forks source link

cry tests converted to add tests. #50

Closed Tikhon03 closed 7 years ago

Tikhon03 commented 7 years ago

Two more add tests written to check partial carrying. Carrying function changed to get tests to pass.

You may want to refactor the carrying function to make it more concise.

arcfide commented 7 years ago

Looks okay, I like the removal of N-wise reduction for basic scalar operations, but the code needs cleanup. Also, you didn't pull in the latest stuff before you did the pull request. Thanks!