Consider a simpler signature for `zxcvbn`
vks opened this issue · 1 comments
vks commented
user_inputs: Option<&[&str]>
is redundant, it could be &[&str]
instead. If there are no user inputs, you would just pass &[]
instead of None
.
shssoichiro commented
I agree 100%. This was just poor thinking on my part.