From d5f2a5d7baa831af6a246e2db63585d2a989ff1f Mon Sep 17 00:00:00 2001 From: Dragyx <66752602+Dragyx@users.noreply.github.com> Date: Sun, 4 May 2025 19:28:31 +0200 Subject: [PATCH] src/main.rs: small refactor for readability --- src/main.rs | 34 +++++++++++++++++----------------- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/src/main.rs b/src/main.rs index 7967981..27e3d22 100644 --- a/src/main.rs +++ b/src/main.rs @@ -74,8 +74,8 @@ fn main() { .collect(); // Compare the package names of both versions - let pre_keys: HashSet = pre.clone().into_keys().collect(); - let post_keys: HashSet = post.clone().into_keys().collect(); + let pre_keys: HashSet = pre.keys().map(|k| k.clone()).collect(); + let post_keys: HashSet = post.keys().map(|k| k.clone()).collect(); // get the intersection of the package names for version changes let maybe_changed: HashSet<_> = pre_keys.intersection(&post_keys).collect(); @@ -117,22 +117,22 @@ fn main() { if p.is_empty() { continue; } - let version_pre = pre.get(p); - let version_post = post.get(p); - if let (Some(ver_pre), Some(ver_post)) = (version_pre, version_post) { - if ver_pre != ver_post { - // println!("C: {p} @ {ver_pre} -> {ver_post}"); - println!( - "{} {} {} {} {} {}", - "[C:]".purple().bold(), - p, - "@".yellow(), - ver_pre.yellow(), - "~>".purple(), - ver_post.cyan() - ); - } + // can not fail since maybe_changed is the union of the keys of pre and post + let ver_pre = pre.get(p).unwrap(); + let ver_post = post.get(p).unwrap(); + + if ver_pre != ver_post { + // println!("C: {p} @ {ver_pre} -> {ver_post}"); + println!( + "{} {} {} {} {} {}", + "[C:]".purple().bold(), + p, + "@".yellow(), + ver_pre.yellow(), + "~>".purple(), + ver_post.cyan() + ); } } if args.closure_size {