search

dalek-cryptography / curve25519-dalek

A pure-Rust implementation of group operations on Ristretto and Curve25519
Other
884 stars 448 forks source link

docs: update `VartimeRistrettoPrecomputation` documentation #667

Open AaronFeickert opened 3 months ago

AaronFeickert commented 3 months ago

It is the case that performing variable-time Ristretto multiscalar multiplication evaluation using VartimeRistrettoPrecomputation may be less efficient than the corresponding VartimeMultiscalarMul implementation, depending on the number of points used.

This PR updates the documentation to make this clear, since the decision of whether or not to use precomputation may depend highly on the use case.

rozbb commented 2 months ago

Nice! I can't quite tell what the formatting error is here

AaronFeickert commented 2 months ago

Looks like the formatter doesn't like it when an empty comment line includes a trailing space. Fixed.