Commit 7ee72183 authored by Rene Saarsoo's avatar Rene Saarsoo
Browse files

Make SourceFileParser into non-singleton class. 

This premature optimization was causing a strange test failure
regarding MetaTagRegistry.  I did some speed measurements and
there seemed to be no performance benefit of having this class
being singleton.
parent 79b59ea9

lib/jsduck/source_file.rb

+1 −1
Original line number Diff line number Diff line
@@ -19,7 +19,7 @@ module JsDuck 
      @html_filename = ""

      @links = {}



      @docs = SourceFileParser.instance.parse(@contents, @filename, options)

      @docs = SourceFileParser.new.parse(@contents, @filename, options)



      @docs.map do |docset|

        link(docset[:linenr], docset)

lib/jsduck/source_file_parser.rb

+0 −2
Original line number Diff line number Diff line 
require 'singleton'

require 'jsduck/js_parser'

require 'jsduck/css_parser'

require 'jsduck/doc_parser'
@@ -15,7 +14,6 @@ module JsDuck 
  # This is the class that brings together all the different steps of

  # parsing the source.

  class SourceFileParser

    include Singleton



    def initialize

      @doc_type = DocType.new