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/update_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/update_command.rs')
| -rw-r--r-- | crates/shirabe/src/command/update_command.rs | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/crates/shirabe/src/command/update_command.rs b/crates/shirabe/src/command/update_command.rs index 4581ce1..42622bd 100644 --- a/crates/shirabe/src/command/update_command.rs +++ b/crates/shirabe/src/command/update_command.rs @@ -308,9 +308,9 @@ impl Command for UpdateCommand { !in_array( PhpMixed::String(package.clone()), &PhpMixed::List(vec![ - Box::new(PhpMixed::String("lock".to_string())), - Box::new(PhpMixed::String("nothing".to_string())), - Box::new(PhpMixed::String("mirrors".to_string())), + PhpMixed::String("lock".to_string()), + PhpMixed::String("nothing".to_string()), + PhpMixed::String("mirrors".to_string()), ]), true, ) @@ -700,9 +700,7 @@ impl UpdateCommand { let mut table = Table::new(output); table.set_headers(vec![PhpMixed::String("Selected packages".to_string())]); for package in &packages { - table.add_row(PhpMixed::List(vec![Box::new(PhpMixed::String( - package.clone(), - ))])); + table.add_row(PhpMixed::List(vec![PhpMixed::String(package.clone())])); } table.render(); |
