From 432472808051cb4f1bb9517b858dbc810aaa5a63 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Sat, 25 Jul 2026 16:16:33 +0900 Subject: refactor: replace redundant clones with moves Co-Authored-By: Claude Opus 5 --- crates/shirabe/src/command/remove_command.rs | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) (limited to 'crates/shirabe/src/command/remove_command.rs') diff --git a/crates/shirabe/src/command/remove_command.rs b/crates/shirabe/src/command/remove_command.rs index 776fca79..eaf9ed41 100644 --- a/crates/shirabe/src/command/remove_command.rs +++ b/crates/shirabe/src/command/remove_command.rs @@ -268,10 +268,7 @@ impl Command for RemoveCommand { } } - let unused: Vec = locked_packages - .iter() - .map(|p| p.get_name().to_string()) - .collect(); + let unused: Vec = locked_packages.iter().map(|p| p.get_name()).collect(); packages.extend(unused); if packages.is_empty() { @@ -485,11 +482,8 @@ impl Command for RemoveCommand { if dry_run { let root_package = composer.get_package(); let mut links: IndexMap> = IndexMap::new(); - links.insert("require".to_string(), root_package.get_requires().clone()); - links.insert( - "require-dev".to_string(), - root_package.get_dev_requires().clone(), - ); + links.insert("require".to_string(), root_package.get_requires()); + links.insert("require-dev".to_string(), root_package.get_dev_requires()); for (link_type, names) in &to_remove { for name in names { if let Some(section) = links.get_mut(link_type.as_str()) { -- cgit v1.3.1