diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-07-25 16:16:33 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-07-25 16:17:10 +0900 |
| commit | 432472808051cb4f1bb9517b858dbc810aaa5a63 (patch) | |
| tree | 4c58b97942853ea2c3f58368203fa93187746cf8 /crates/shirabe/src/dependency_resolver/rule.rs | |
| parent | d4608662f28b9a5135986b1702afe3199957eabe (diff) | |
| download | php-shirabe-432472808051cb4f1bb9517b858dbc810aaa5a63.tar.gz php-shirabe-432472808051cb4f1bb9517b858dbc810aaa5a63.tar.zst php-shirabe-432472808051cb4f1bb9517b858dbc810aaa5a63.zip | |
refactor: replace redundant clones with moves
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe/src/dependency_resolver/rule.rs')
| -rw-r--r-- | crates/shirabe/src/dependency_resolver/rule.rs | 18 |
1 files changed, 4 insertions, 14 deletions
diff --git a/crates/shirabe/src/dependency_resolver/rule.rs b/crates/shirabe/src/dependency_resolver/rule.rs index e1572b0b..4ed8619c 100644 --- a/crates/shirabe/src/dependency_resolver/rule.rs +++ b/crates/shirabe/src/dependency_resolver/rule.rs @@ -146,7 +146,7 @@ impl Rule { _ => None, }, r if r == RULE_FIXED => match self.get_reason_data() { - ReasonData::Fixed { package } => Some(package.get_name().to_string()), + ReasonData::Fixed { package } => Some(package.get_name()), _ => None, }, r if r == RULE_PACKAGE_REQUIRES => match self.get_reason_data() { @@ -216,12 +216,7 @@ impl Rule { return true; } if !link.get_constraint().matches( - &SimpleConstraint::new( - "=".to_string(), - p.get_version().to_string(), - None, - ) - .into(), + &SimpleConstraint::new("=".to_string(), p.get_version(), None).into(), ) { return true; } @@ -256,12 +251,7 @@ impl Rule { return true; } if !constraint.matches( - &SimpleConstraint::new( - "=".to_string(), - p.get_version().to_string(), - None, - ) - .into(), + &SimpleConstraint::new("=".to_string(), p.get_version(), None).into(), ) { return true; } @@ -475,7 +465,7 @@ impl Rule { requires.push(pool.literal_to_package(*literal)); } - let text = link.get_pretty_string(source_package.clone()); + let text = link.get_pretty_string(source_package); if !requires.is_empty() { format!( "{} -> satisfiable by {}.", |