diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-06-20 18:34:54 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-06-20 18:34:54 +0900 |
| commit | 81b9fc9d92bb74aa8428ae4db39bd84e8c16095c (patch) | |
| tree | 3efb6476d797e2a95545c4c3abba468c3e3c8d52 /crates/shirabe/src/command/repository_command.rs | |
| parent | c09cd630afb4bb0ca10e926f93bf706ca828ae85 (diff) | |
| download | php-shirabe-81b9fc9d92bb74aa8428ae4db39bd84e8c16095c.tar.gz php-shirabe-81b9fc9d92bb74aa8428ae4db39bd84e8c16095c.tar.zst php-shirabe-81b9fc9d92bb74aa8428ae4db39bd84e8c16095c.zip | |
refactor(php-shim): drop Box wrapping from PhpMixed List/Array
The List and Array variants of PhpMixed boxed their elements
unnecessarily. Store PhpMixed values directly and update all callers
accordingly.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe/src/command/repository_command.rs')
| -rw-r--r-- | crates/shirabe/src/command/repository_command.rs | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/crates/shirabe/src/command/repository_command.rs b/crates/shirabe/src/command/repository_command.rs index 6ea26d2..caca808 100644 --- a/crates/shirabe/src/command/repository_command.rs +++ b/crates/shirabe/src/command/repository_command.rs @@ -84,7 +84,7 @@ impl RepositoryCommand { } if !packagist_present { let mut packagist_entry = IndexMap::new(); - packagist_entry.insert("packagist.org".to_string(), Box::new(PhpMixed::Bool(false))); + packagist_entry.insert("packagist.org".to_string(), PhpMixed::Bool(false)); repos.insert(repos.len().to_string(), PhpMixed::Array(packagist_entry)); } @@ -102,7 +102,7 @@ impl RepositoryCommand { if let PhpMixed::Array(ref repo_map) = *repo { if repo_map.len() == 1 && let Some(first_val) = repo_map.values().next() - && matches!(**first_val, PhpMixed::Bool(false)) + && matches!(*first_val, PhpMixed::Bool(false)) { let first_key = repo_map.keys().next().unwrap(); io.write(&format!("[{}] <info>disabled</info>", first_key)); @@ -273,7 +273,7 @@ impl Command for RepositoryCommand { .get_path() .to_string(); let config_data_map: IndexMap<String, PhpMixed> = match config_data { - PhpMixed::Array(m) => m.into_iter().map(|(k, v)| (k, *v)).collect(), + PhpMixed::Array(m) => m.into_iter().collect(), _ => IndexMap::new(), }; self.config @@ -312,11 +312,8 @@ impl Command for RepositoryCommand { })); } let mut m = IndexMap::new(); - m.insert( - "type".to_string(), - Box::new(PhpMixed::String(arg1_str.to_string())), - ); - m.insert("url".to_string(), Box::new(PhpMixed::String(arg2.unwrap()))); + m.insert("type".to_string(), PhpMixed::String(arg1_str.to_string())); + m.insert("url".to_string(), PhpMixed::String(arg2.unwrap())); PhpMixed::Array(m) }; |
