diff options
author | one230six <723682061@qq.com> | 2024-03-13 15:18:12 +0800 |
---|---|---|
committer | David Peter <sharkdp@users.noreply.github.com> | 2024-03-14 08:46:35 +0100 |
commit | c0921351e7eb9814fc829dbf48a5c745c4b0ad91 (patch) | |
tree | f99c84e03b4d081e445e839aea3978d981630bb7 | |
parent | 865b49609856593b91223d7bfa8b18d358361296 (diff) |
refactor: Optimize code based on cargo clippy suggestions
Signed-off-by: one230six <723682061@qq.com>
refactor: Optimize code based on cargo clippy suggestions
Signed-off-by: one230six <723682061@qq.com>
-rw-r--r-- | src/command.rs | 10 | ||||
-rw-r--r-- | src/parameter/mod.rs | 10 | ||||
-rw-r--r-- | src/util/min_max.rs | 2 |
3 files changed, 11 insertions, 11 deletions
diff --git a/src/command.rs b/src/command.rs index 55cbb5e..f580691 100644 --- a/src/command.rs +++ b/src/command.rs @@ -59,11 +59,11 @@ impl<'a> Command<'a> { } pub fn get_name_with_unused_parameters(&self) -> String { - let parameters = - self.get_unused_parameters() - .fold(String::new(), |output, (parameter, value)| { - output + &format!("{} = {}, ", parameter, value.to_string()) - }); + let parameters = self + .get_unused_parameters() + .fold(String::new(), |output, (parameter, value)| { + output + &format!("{} = {}, ", parameter, value) + }); let parameters = parameters.trim_end_matches(", "); let parameters = if parameters.is_empty() { "".into() diff --git a/src/parameter/mod.rs b/src/parameter/mod.rs index 994f4a2..c766713 100644 --- a/src/parameter/mod.rs +++ b/src/parameter/mod.rs @@ -1,4 +1,5 @@ use crate::util::number::Number; +use std::fmt::Display; pub mod range_step; pub mod tokenize; @@ -9,12 +10,13 @@ pub enum ParameterValue { Numeric(Number), } -impl ToString for ParameterValue { - fn to_string(&self) -> String { - match self { +impl Display for ParameterValue { + fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result { + let str = match self { ParameterValue::Text(ref value) => value.clone(), ParameterValue::Numeric(value) => value.to_string(), - } + }; + write!(f, "{}", str) } } diff --git a/src/util/min_max.rs b/src/util/min_max.rs index 9e11f55..d4ee07e 100644 --- a/src/util/min_max.rs +++ b/src/util/min_max.rs @@ -1,5 +1,3 @@ -use std::iter::Iterator; - /// A max function for f64's without NaNs pub fn max(vals: &[f64]) -> f64 { *vals |