Skip to content

Conversation

@tarcieri
Copy link
Member

...and remove the now legacy FixedBaseScalarMul trait.

This commit impls the changes from RustCrypto/traits#229 and RustCrypto/traits#230:

  • Adds a FromSecretKey impl on Scalar
  • Adds a FromPublicKey impl on AffinePoint
  • Adds MulBase impl on Scalar

@codecov-commenter
Copy link

codecov-commenter commented Jul 29, 2020

Codecov Report

Merging #100 into master will decrease coverage by 0.17%.
The diff coverage is 77.77%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #100      +/-   ##
==========================================
- Coverage   52.83%   52.66%   -0.18%     
==========================================
  Files          19       19              
  Lines        3435     3437       +2     
==========================================
- Hits         1815     1810       -5     
- Misses       1620     1627       +7     
Impacted Files Coverage Δ
p256/src/arithmetic/scalar.rs 62.98% <0.00%> (-0.83%) ⬇️
k256/src/arithmetic/scalar.rs 72.81% <50.00%> (-1.37%) ⬇️
k256/src/arithmetic.rs 85.95% <100.00%> (-0.15%) ⬇️
p256/src/arithmetic.rs 85.31% <100.00%> (+0.88%) ⬆️
k256/src/mul.rs 83.72% <0.00%> (-4.66%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0381cd6...89da786. Read the comment docs.

...and remove the now legacy `FixedBaseScalarMul` trait.

This commit impls the changes from RustCrypto/traits#229 and
RustCrypto/traits#230:

- Adds a `FromSecretKey` impl on `Scalar`
- Adds a `FromPublicKey` impl on `AffinePoint`
- Adds `MulBase` impl on `Scalar`
@tarcieri tarcieri force-pushed the from-public-secret-key-and-mul-base branch from 605d36d to 89da786 Compare July 29, 2020 17:35
@tarcieri tarcieri merged commit c6fae25 into master Jul 29, 2020
@tarcieri tarcieri deleted the from-public-secret-key-and-mul-base branch July 29, 2020 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants