From 34facc246d0d439e9329364622ec256d5f0f29bc Mon Sep 17 00:00:00 2001 From: OMGeeky Date: Wed, 8 May 2024 18:25:36 +0200 Subject: [PATCH] remove clone if not needed in shortener --- src/client/youtube/data.rs | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/client/youtube/data.rs b/src/client/youtube/data.rs index 638c156..3a71572 100644 --- a/src/client/youtube/data.rs +++ b/src/client/youtube/data.rs @@ -157,10 +157,11 @@ fn substitute_common( .replace(PART_COUNT, &max.to_string())) } -fn shorten_string_if_needed(s: &str, target_len: Option) -> String { +fn shorten_string_if_needed(s: impl Into, target_len: Option) -> String { + let s = s.into(); const SHORTEN_CHARS: &str = "..."; if target_len.is_none() { - return s.to_string(); + return s; } let target_len = target_len.unwrap(); if target_len < SHORTEN_CHARS.len() { @@ -172,7 +173,7 @@ fn shorten_string_if_needed(s: &str, target_len: Option) -> String { assert_eq!(result.len(), target_len); result } else { - s.to_string() + s } } fn get_date_prefix(date: chrono::NaiveDate) -> String {