diff options
| author | Samuel Giddins <segiddins@segiddins.me> | 2023-11-27 16:06:27 -0600 |
|---|---|---|
| committer | git <svn-admin@ruby-lang.org> | 2023-12-01 19:50:03 +0000 |
| commit | 0ed55bf09713299201caec3ba9d9ef7ba6e8526f (patch) | |
| tree | 4235b408a02137787a6af425b6695a965f1fe8f2 /lib/bundler/dependency.rb | |
| parent | a607d62d8c5786087916413c369dbde0c62db573 (diff) | |
[rubygems/rubygems] Reduce array allocations when loading definition
The same array was being re-created in a loop (as well as the `generic_local_platform`), which is avoidable by hoisting it to a frozen array created once
https://github.com/rubygems/rubygems/commit/009a3c6d0d
Diffstat (limited to 'lib/bundler/dependency.rb')
| -rw-r--r-- | lib/bundler/dependency.rb | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/lib/bundler/dependency.rb b/lib/bundler/dependency.rb index 21a4564dcc..82c4c1866b 100644 --- a/lib/bundler/dependency.rb +++ b/lib/bundler/dependency.rb @@ -48,10 +48,13 @@ module Bundler @autorequire = Array(options["require"] || []) if options.key?("require") end + RUBY_PLATFORM_ARRAY = [Gem::Platform::RUBY].freeze + private_constant :RUBY_PLATFORM_ARRAY + # Returns the platforms this dependency is valid for, in the same order as # passed in the `valid_platforms` parameter def gem_platforms(valid_platforms) - return [Gem::Platform::RUBY] if force_ruby_platform + return RUBY_PLATFORM_ARRAY if force_ruby_platform return valid_platforms if @platforms.empty? valid_platforms.select {|p| expanded_platforms.include?(GemHelpers.generic(p)) } |