From 271fb1f73db29f9fccbe4aaf8108989a0bb04ae2 Mon Sep 17 00:00:00 2001 From: discord9 Date: Tue, 3 Feb 2026 20:10:35 +0800 Subject: [PATCH] clippy Signed-off-by: discord9 --- tests/runner/src/cmd/compat.rs | 8 ++++---- tests/runner/src/version.rs | 5 +++-- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/tests/runner/src/cmd/compat.rs b/tests/runner/src/cmd/compat.rs index 9a6a38f9aa..f4845ce291 100644 --- a/tests/runner/src/cmd/compat.rs +++ b/tests/runner/src/cmd/compat.rs @@ -47,7 +47,7 @@ impl CompatCommand { let from_version = match Version::parse(&self.from) { Ok(v) => v, Err(e) => { - eprintln!("Error parsing 'from' version: {}", e); + println!("Error parsing 'from' version: {}", e); std::process::exit(1); } }; @@ -55,7 +55,7 @@ impl CompatCommand { let to_version = match Version::parse(&self.to) { Ok(v) => v, Err(e) => { - eprintln!("Error parsing 'to' version: {}", e); + println!("Error parsing 'to' version: {}", e); std::process::exit(1); } }; @@ -78,7 +78,7 @@ impl CompatCommand { { Ok(r) => r, Err(e) => { - eprintln!("Failed to create compatibility runner: {}", e); + println!("Failed to create compatibility runner: {}", e); std::process::exit(1); } }; @@ -88,7 +88,7 @@ impl CompatCommand { println!("\x1b[32mCompatibility tests passed!\x1b[0m"); } Err(e) => { - eprintln!("\x1b[31mCompatibility tests failed: {}\x1b[0m", e); + println!("\x1b[31mCompatibility tests failed: {}\x1b[0m", e); std::process::exit(1); } } diff --git a/tests/runner/src/version.rs b/tests/runner/src/version.rs index 6c809e992f..f2eb9e081a 100644 --- a/tests/runner/src/version.rs +++ b/tests/runner/src/version.rs @@ -54,11 +54,12 @@ impl Version { if lower == "current" { Ok(Version::Current) } else { - let inner = SemverVersion::parse(s).map_err(|e| VersionError::ParseError(e))?; + let inner = SemverVersion::parse(s).map_err(VersionError::ParseError)?; Ok(Version::Semantic(inner)) } } + #[allow(unused)] pub fn is_current(&self) -> bool { matches!(self, Version::Current) } @@ -131,7 +132,7 @@ impl Version { impl PartialOrd for Version { fn partial_cmp(&self, other: &Self) -> Option { - Some(self.compare(other)) + Some(Ord::cmp(self, other)) } }