Add missing Clippy allows to solutions

This commit is contained in:
mo8it 2024-08-22 14:41:25 +02:00
parent 423b50b068
commit f1abd8577c
2 changed files with 4 additions and 4 deletions

View file

@ -18,12 +18,11 @@ fn main() {
// Here, both answers work. // Here, both answers work.
// `.into()` converts a type into an expected type. // `.into()` converts a type into an expected type.
// If it is called where `String` is expected, it will convert `&str` to `String`. // If it is called where `String` is expected, it will convert `&str` to `String`.
// But if is called where `&str` is expected, then `&str` is kept `&str` since no
// conversion is needed.
string("nice weather".into()); string("nice weather".into());
// But if it is called where `&str` is expected, then `&str` is kept `&str` since no conversion is needed.
// If you remove the `#[allow(…)]` line, then Clippy will tell you to remove `.into()` below since it is a useless conversion.
#[allow(clippy::useless_conversion)]
string_slice("nice weather".into()); string_slice("nice weather".into());
// ^^^^^^^ the compiler recommends removing the `.into()`
// call because it is a useless conversion.
string(format!("Interpolation {}", "Station")); string(format!("Interpolation {}", "Station"));

View file

@ -25,6 +25,7 @@ fn factorial_fold(num: u64) -> u64 {
// -> 1 * 2 is calculated, then the result 2 is multiplied by // -> 1 * 2 is calculated, then the result 2 is multiplied by
// the second element 3 so the result 6 is returned. // the second element 3 so the result 6 is returned.
// And so on… // And so on…
#[allow(clippy::unnecessary_fold)]
(2..=num).fold(1, |acc, x| acc * x) (2..=num).fold(1, |acc, x| acc * x)
} }