search

rozbb / saber-rs

A pure-Rust implementation of the Saber key encapsulation mechanism (KEM)
Other
4 stars 1 forks source link

Nit: `unwrap()` may or may not cause panics #2

Closed pinkforest closed 3 weeks ago

pinkforest commented 3 weeks ago

There is some use of unwrap() that may need to be cleaned / documented. Some may be infallible behind try_into()

rozbb commented 3 weeks ago

Resolved in fe9c1aff751004210d6e60abce37d308068fe43e and the subsequent 2 commits. All the unwraps in the library are now expect, and are infallible