Remove unnecessary file_name parameter from Parser#initialize (#1146

)

Similar to #1135, the `file_name` argument's value should be the name as
`top_level.absolute_name`. So passing it separately is not necessary.

lib/rdoc/parser.rb

@@ -191,7 +191,7 @@ def self.for top_level, content, options, stats
 
     content = remove_modeline content
 
-    parser.new top_level, file_name, content, options, stats
+    parser.new top_level, content, options, stats
   rescue SystemCallError
     nil
   end
@@ -252,12 +252,12 @@ def self.use_markup content
   # RDoc::Markup::PreProcess object is created which allows processing of
   # directives.
 
-  def initialize top_level, file_name, content, options, stats
+  def initialize top_level, content, options, stats
     @top_level = top_level
     @top_level.parser = self.class
     @store = @top_level.store
 
-    @file_name = file_name
+    @file_name = top_level.absolute_name
     @content = content
     @options = options
     @stats = stats

lib/rdoc/parser/c.rb

@@ -168,7 +168,7 @@ class RDoc::Parser::C < RDoc::Parser
   # Prepares for parsing a C file.  See RDoc::Parser#initialize for details on
   # the arguments.
 
-  def initialize top_level, file_name, content, options, stats
+  def initialize top_level, content, options, stats
     super
 
     @known_classes = RDoc::KNOWN_CLASSES.dup

lib/rdoc/parser/prism_ruby.rb

@@ -18,7 +18,7 @@ class RDoc::Parser::PrismRuby < RDoc::Parser
   attr_accessor :visibility
   attr_reader :container, :singleton
 
-  def initialize(top_level, file_name, content, options, stats)
+  def initialize(top_level, content, options, stats)
     super
 
     content = handle_tab_width(content)

lib/rdoc/parser/ruby.rb

@@ -170,7 +170,7 @@ class RDoc::Parser::Ruby < RDoc::Parser
   ##
   # Creates a new Ruby parser.
 
-  def initialize(top_level, file_name, content, options, stats)
+  def initialize(top_level, content, options, stats)
     super
 
     content = handle_tab_width(content)

lib/rdoc/parser/simple.rb

@@ -14,7 +14,7 @@ class RDoc::Parser::Simple < RDoc::Parser
   ##
   # Prepare to parse a plain file
 
-  def initialize(top_level, file_name, content, options, stats)
+  def initialize(top_level, content, options, stats)
     super
 
     preprocess = RDoc::Markup::PreProcess.new @file_name, @options.rdoc_include

test/rdoc/test_rdoc_parser.rb

@@ -310,7 +310,7 @@ def test_class_use_markup_unknown
 
   def test_initialize
     with_top_level("file.rb", "") do |top_level, content|
-      @RP.new top_level, top_level.absolute_name, content, @options, nil
+      @RP.new top_level, content, @options, nil
 
       assert_equal @RP, top_level.parser
     end

test/rdoc/test_rdoc_parser_c.rb

@@ -138,7 +138,7 @@ def test_initialize
       @fn => { 'cSomeExtSingle' => 'SomeExtSingle' }
     }
 
-    parser = RDoc::Parser::C.new @top_level, @fn, '', @options, @stats
+    parser = RDoc::Parser::C.new @top_level, '', @options, @stats
 
     expected = { 'cSomeExt' => some_ext }
     assert_equal expected, parser.classes
@@ -2144,7 +2144,7 @@ def util_get_class content, name = nil
   end
 
   def util_parser content = ''
-    RDoc::Parser::C.new @top_level, @fn, content, @options, @stats
+    RDoc::Parser::C.new @top_level, content, @options, @stats
   end
 
 end

test/rdoc/test_rdoc_parser_changelog.rb

@@ -474,8 +474,7 @@ def test_scan_git_commit_date
   end
 
   def util_parser content = ''
-    RDoc::Parser::ChangeLog.new \
-      @top_level, @tempfile.path, content, @options, @stats
+    RDoc::Parser::ChangeLog.new @top_level, content, @options, @stats
   end
 
   def log_entry(*a)

test/rdoc/test_rdoc_parser_markdown.rb

@@ -55,7 +55,7 @@ def test_scan
   end
 
   def util_parser content
-    RDoc::Parser::Markdown.new @top_level, @fn, content, @options, @stats
+    RDoc::Parser::Markdown.new @top_level, content, @options, @stats
   end
 
 end

test/rdoc/test_rdoc_parser_prism_ruby.rb

@@ -1996,7 +1996,7 @@ def accept_legacy_bug?
   end
 
   def util_parser(content)
-    @parser = RDoc::Parser::PrismRuby.new @top_level, @filename, content, @options, @stats
+    @parser = RDoc::Parser::PrismRuby.new @top_level, content, @options, @stats
     @parser.scan
   end
 end
@@ -2010,7 +2010,7 @@ def accept_legacy_bug?
   end
 
   def util_parser(content)
-    @parser = RDoc::Parser::Ruby.new @top_level, @filename, content, @options, @stats
+    @parser = RDoc::Parser::Ruby.new @top_level, content, @options, @stats
     @parser.scan
   end
 end unless ENV['RDOC_USE_PRISM_PARSER']

test/rdoc/test_rdoc_parser_rd.rb

@@ -49,7 +49,7 @@ def test_scan
   end
 
   def util_parser content
-    RDoc::Parser::RD.new @top_level, @fn, content, @options, @stats
+    RDoc::Parser::RD.new @top_level, content, @options, @stats
   end
 
 end

test/rdoc/test_rdoc_parser_ruby.rb

@@ -4087,8 +4087,7 @@ class Baz
   end
 
   def util_parser(content)
-    @parser = RDoc::Parser::Ruby.new @top_level, @filename, content, @options,
-                                     @stats
+    @parser = RDoc::Parser::Ruby.new @top_level, content, @options, @stats
   end
 
   def util_two_parsers(first_file_content, second_file_content)

test/rdoc/test_rdoc_parser_simple.rb

@@ -109,7 +109,7 @@ def test_scan
   end
 
   def util_parser(content)
-    RDoc::Parser::Simple.new @top_level, @fn, content, @options, @stats
+    RDoc::Parser::Simple.new @top_level, content, @options, @stats
   end
 
 end

test/rdoc/test_rdoc_store.rb

@@ -116,7 +116,7 @@ def test_add_c_variables
     some_ext   = c_file.add_class RDoc::NormalClass, 'SomeExt'
                  c_file.add_class RDoc::SingleClass, 'SomeExtSingle'
 
-    c_parser = RDoc::Parser::C.new c_file, 'ext.c', '', options, nil
+    c_parser = RDoc::Parser::C.new c_file, '', options, nil
 
     c_parser.classes['cSomeExt']             = some_ext
     c_parser.singleton_classes['s_cSomeExt'] = 'SomeExtSingle'
@@ -698,7 +698,7 @@ def test_save_cache
     some_ext   = c_file.add_class RDoc::NormalClass, 'SomeExt'
                  c_file.add_class RDoc::SingleClass, 'SomeExtSingle'
 
-    c_parser = RDoc::Parser::C.new c_file, 'ext.c', '', options, nil
+    c_parser = RDoc::Parser::C.new c_file, '', options, nil
 
     c_parser.classes['cSomeExt']             = some_ext
     c_parser.singleton_classes['s_cSomeExt'] = 'SomeExtSingle'

test/rdoc/xref_test_case.rb

@@ -20,8 +20,7 @@ def setup
 
     stats = RDoc::Stats.new @store, 0
 
-    parser = RDoc::Parser::Ruby.new @xref_data, @file_name, XREF_DATA, @options,
-                                    stats
+    parser = RDoc::Parser::Ruby.new @xref_data, XREF_DATA, @options, stats
 
     @example_md = @store.add_file 'EXAMPLE.md'
     @example_md.parser = RDoc::Parser::Markdown