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/autoload/class_loader.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/autoload/class_loader.rs')
| -rw-r--r-- | crates/shirabe/src/autoload/class_loader.rs | 24 |
1 files changed, 10 insertions, 14 deletions
diff --git a/crates/shirabe/src/autoload/class_loader.rs b/crates/shirabe/src/autoload/class_loader.rs index 918b319..b19e618 100644 --- a/crates/shirabe/src/autoload/class_loader.rs +++ b/crates/shirabe/src/autoload/class_loader.rs @@ -495,11 +495,7 @@ impl ClassLoader { pub fn as_array_iter(&self) -> Vec<(String, PhpMixed)> { let key = |name: &str| format!("\0Composer\\Autoload\\ClassLoader\0{}", name); let str_list = |v: &Vec<String>| { - PhpMixed::List( - v.iter() - .map(|s| Box::new(PhpMixed::String(s.clone()))) - .collect(), - ) + PhpMixed::List(v.iter().map(|s| PhpMixed::String(s.clone())).collect()) }; vec![ @@ -518,12 +514,12 @@ impl ClassLoader { .map(|(k, inner)| { ( k.clone(), - Box::new(PhpMixed::Array( + PhpMixed::Array( inner .iter() - .map(|(k2, n)| (k2.clone(), Box::new(PhpMixed::Int(*n)))) + .map(|(k2, n)| (k2.clone(), PhpMixed::Int(*n))) .collect(), - )), + ), ) }) .collect(), @@ -534,7 +530,7 @@ impl ClassLoader { PhpMixed::Array( self.prefix_dirs_psr4 .iter() - .map(|(k, v)| (k.clone(), Box::new(str_list(v)))) + .map(|(k, v)| (k.clone(), str_list(v))) .collect(), ), ), @@ -547,12 +543,12 @@ impl ClassLoader { .map(|(k, inner)| { ( k.clone(), - Box::new(PhpMixed::Array( + PhpMixed::Array( inner .iter() - .map(|(k2, v)| (k2.clone(), Box::new(str_list(v)))) + .map(|(k2, v)| (k2.clone(), str_list(v))) .collect(), - )), + ), ) }) .collect(), @@ -565,7 +561,7 @@ impl ClassLoader { PhpMixed::Array( self.class_map .iter() - .map(|(k, v)| (k.clone(), Box::new(PhpMixed::String(v.clone())))) + .map(|(k, v)| (k.clone(), PhpMixed::String(v.clone()))) .collect(), ), ), @@ -578,7 +574,7 @@ impl ClassLoader { PhpMixed::Array( self.missing_classes .iter() - .map(|(k, b)| (k.clone(), Box::new(PhpMixed::Bool(*b)))) + .map(|(k, b)| (k.clone(), PhpMixed::Bool(*b))) .collect(), ), ), |