Skip to content

Commit

Permalink
remove explicit require(s) for auto-loaded constant(s)
Browse files Browse the repository at this point in the history
the Gem module's auto-loads will handle loading these as needed,

this started as a redundancy found in *rubygems.rb* which had:
`autoload :Specification, 'rubygems/specification'` as well as
`require 'rubygems/specification'`
  • Loading branch information
kares committed Jun 27, 2020
1 parent 3c7f249 commit 43ceae7
Show file tree
Hide file tree
Showing 7 changed files with 0 additions and 13 deletions.
2 changes: 0 additions & 2 deletions lib/rubygems.rb
Original file line number Diff line number Diff line change
Expand Up @@ -1335,8 +1335,6 @@ def default_gem_load_paths
autoload :Specification, File.expand_path('rubygems/specification', __dir__)
autoload :Util, File.expand_path('rubygems/util', __dir__)
autoload :Version, File.expand_path('rubygems/version', __dir__)

require "rubygems/specification"
end

require 'rubygems/exceptions'
Expand Down
3 changes: 0 additions & 3 deletions lib/rubygems/dependency.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,6 @@
##
# The Dependency class holds a Gem name and a Gem::Requirement.

require "rubygems/bundler_version_finder"
require "rubygems/requirement"

class Gem::Dependency

##
Expand Down
1 change: 0 additions & 1 deletion lib/rubygems/gem_runner.rb
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@

require 'rubygems'
require 'rubygems/command_manager'
require 'rubygems/config_file'
require 'rubygems/deprecate'

##
Expand Down
2 changes: 0 additions & 2 deletions lib/rubygems/name_tuple.rb
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@
# Represents a gem of name +name+ at +version+ of +platform+. These
# wrap the data returned from the indexes.

require 'rubygems/platform'

class Gem::NameTuple

def initialize(name, version, platform="ruby")
Expand Down
1 change: 0 additions & 1 deletion lib/rubygems/package.rb
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@

require "rubygems"
require 'rubygems/security'
require 'rubygems/specification'
require 'rubygems/user_interaction'
require 'zlib'

Expand Down
1 change: 0 additions & 1 deletion lib/rubygems/requirement.rb
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
# frozen_string_literal: true
require "rubygems/version"
require "rubygems/deprecate"

##
Expand Down
3 changes: 0 additions & 3 deletions lib/rubygems/specification.rb
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,6 @@
# See LICENSE.txt for permissions.
#++

require 'rubygems/version'
require 'rubygems/requirement'
require 'rubygems/platform'
require 'rubygems/deprecate'
require 'rubygems/basic_specification'
require 'rubygems/stub_specification'
Expand Down

0 comments on commit 43ceae7

Please sign in to comment.