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/repository/filesystem_repository.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/repository/filesystem_repository.rs')
| -rw-r--r-- | crates/shirabe/src/repository/filesystem_repository.rs | 22 |
1 files changed, 6 insertions, 16 deletions
diff --git a/crates/shirabe/src/repository/filesystem_repository.rs b/crates/shirabe/src/repository/filesystem_repository.rs index 32fbcfd7..d4b9b72e 100644 --- a/crates/shirabe/src/repository/filesystem_repository.rs +++ b/crates/shirabe/src/repository/filesystem_repository.rs @@ -460,7 +460,7 @@ impl FilesystemRepository { "root".to_string(), PhpMixed::Array( self.dump_root_package( - current_root.clone(), + current_root, install_paths, dev_mode, repo_dir, @@ -602,11 +602,7 @@ impl FilesystemRepository { // PHP: ($package->getSourceReference() ?: $package->getDistReference()) ?: null; let source = package.get_source_reference().unwrap_or_default(); let dist = package.get_dist_reference().unwrap_or_default(); - let combined = if !source.is_empty() { - source.to_string() - } else { - dist.to_string() - }; + let combined = if !source.is_empty() { source } else { dist }; reference = if combined.is_empty() { None } else { @@ -630,11 +626,11 @@ impl FilesystemRepository { let mut data: IndexMap<String, PhpMixed> = IndexMap::new(); data.insert( "pretty_version".to_string(), - PhpMixed::String(package.get_pretty_version().to_string()), + PhpMixed::String(package.get_pretty_version()), ); data.insert( "version".to_string(), - PhpMixed::String(package.get_version().to_string()), + PhpMixed::String(package.get_version()), ); data.insert( "reference".to_string(), @@ -643,10 +639,7 @@ impl FilesystemRepository { None => PhpMixed::Null, }, ); - data.insert( - "type".to_string(), - PhpMixed::String(package.get_type().to_string()), - ); + data.insert("type".to_string(), PhpMixed::String(package.get_type())); data.insert( "install_path".to_string(), match install_path { @@ -687,10 +680,7 @@ impl FilesystemRepository { ); let mut result: IndexMap<String, PhpMixed> = IndexMap::new(); - result.insert( - "name".to_string(), - PhpMixed::String(package.get_name().to_string()), - ); + result.insert("name".to_string(), PhpMixed::String(package.get_name())); result.insert( "pretty_version".to_string(), data.get("pretty_version") |
