diff options
| author | Jeremy Evans <code@jeremyevans.net> | 2024-05-14 17:03:46 -0700 |
|---|---|---|
| committer | Jeremy Evans <code@jeremyevans.net> | 2024-05-21 05:33:57 +0900 |
| commit | 86cf074fa1dcf73846e094775414d661e47dfc76 (patch) | |
| tree | 3211bf56a224daae8d4dd567c8529a4affdbd306 /test/ruby | |
| parent | 2433b57b6a3a6f8e65f61c27d707e1c7f5407986 (diff) | |
Avoid array allocation for empty ruby2_keywords flagged keyword hash
If the method being called does not have a positional splat
parameter, there is no point in allocating the array, as
decrementing given_argc is sufficient to ensure the empty keyword
hash is not considered an argument, assuming that we are calling
a method/lambda and not a regular proc.
Diffstat (limited to 'test/ruby')
| -rw-r--r-- | test/ruby/test_allocation.rb | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/test/ruby/test_allocation.rb b/test/ruby/test_allocation.rb index ab3ae15a9f..fbe0548899 100644 --- a/test/ruby/test_allocation.rb +++ b/test/ruby/test_allocation.rb @@ -100,7 +100,7 @@ class TestAllocation < Test::Unit::TestCase check_allocations(0, 1, "none(**empty_hash, **empty_hash#{block})") check_allocations(1, 1, "none(*empty_array, *empty_array, **empty_hash, **empty_hash#{block})") - check_allocations(#{block.empty?} ? 0 : 1, 0, "none(*r2k_empty_array#{block})") + check_allocations(0, 0, "none(*r2k_empty_array#{block})") RUBY end @@ -120,7 +120,7 @@ class TestAllocation < Test::Unit::TestCase check_allocations(0, 1, "required(**hash1, **empty_hash#{block})") check_allocations(1, 0, "required(*array1, *empty_array, **empty_hash#{block})") - check_allocations(#{block.empty?} ? 0 : 1, 0, "required(*r2k_empty_array1#{block})") + check_allocations(0, 0, "required(*r2k_empty_array1#{block})") check_allocations(0, 1, "required(*r2k_array#{block})") # Currently allocates 1 array unnecessarily due to splatarray true @@ -144,8 +144,8 @@ class TestAllocation < Test::Unit::TestCase check_allocations(0, 1, "optional(**hash1, **empty_hash#{block})") check_allocations(1, 0, "optional(*array1, *empty_array, **empty_hash#{block})") - check_allocations(#{block.empty?} ? 0 : 1, 0, "optional(*r2k_empty_array#{block})") - check_allocations(#{block.empty?} ? 0 : 1, 0, "optional(*r2k_empty_array1#{block})") + check_allocations(0, 0, "optional(*r2k_empty_array#{block})") + check_allocations(0, 0, "optional(*r2k_empty_array1#{block})") check_allocations(0, 1, "optional(*r2k_array#{block})") # Currently allocates 1 array unnecessarily due to splatarray true |