From b642c34e42bf4505ec49ab774cd28f92813679dd Mon Sep 17 00:00:00 2001 From: Pierre Lalet Date: Wed, 18 Oct 2023 02:18:22 +0200 Subject: [PATCH] Remove unneeded .clone() call `pattern` is a reference, and according to `cargo test` output: "the type `[u8]` does not implement `Clone`, so calling `clone` on `&[u8]` copies the reference, which does not do anything and can be removed". --- src/smack/smack.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/smack/smack.rs b/src/smack/smack.rs index bb37358..8248ec1 100644 --- a/src/smack/smack.rs +++ b/src/smack/smack.rs @@ -61,7 +61,7 @@ pub struct Smack { } fn make_copy_of_pattern(pattern: &[u8], is_nocase: bool) -> Vec { - let mut p = pattern.clone().to_vec(); + let mut p = pattern.to_vec(); for i in 0..p.len() { if is_nocase { p[i] = p[i].to_ascii_lowercase();