You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
#[allow(dead_code)]// this field is used only with the `PasswordHasher` impl
pub(crate)len:usize,
However, doc comments of Params::new and Params::RECOMMENDED_LEN does not mention this. So, I suggest mentioning this in doc comments as well. I think this makes it easier to understand that len parameter of Params::new is used only in PasswordHasher and any value is meaningless when PasswordHasher is not used.
The text was updated successfully, but these errors were encountered:
I know this has already been mentioned in the comment.
password-hashes/scrypt/src/params.rs
Lines 14 to 15 in 7ff9080
However, doc comments of
Params::new
andParams::RECOMMENDED_LEN
does not mention this. So, I suggest mentioning this in doc comments as well. I think this makes it easier to understand thatlen
parameter ofParams::new
is used only inPasswordHasher
and any value is meaningless whenPasswordHasher
is not used.The text was updated successfully, but these errors were encountered: