Commit bd4c3b7c authored by Rene Saarsoo's avatar Rene Saarsoo
Browse files

Use Array() to avoid nils in Process::Lint. 

Just doing things in more conventional way.
parent 341b39cd

lib/jsduck/process/inherit_members.rb

+1 −0
Original line number Diff line number Diff line
@@ -82,6 +82,7 @@ module JsDuck 
        m[:params] = parent[:params] if inherit_params?(m, parent)

        m[:return] = parent[:return] unless m[:return]

        m[:throws] = parent[:throws] unless m[:throws] && m[:throws].length > 0

        m[:properties] = parent[:properties] unless m[:properties] && m[:properties].length > 0



        # Don't inherit type from parent when:

        # - member itself has type and it's not auto-detected

lib/jsduck/process/lint.rb

+2 −2
Original line number Diff line number Diff line
@@ -27,7 +27,7 @@ module JsDuck 
          if !member[:name] || member[:name] == ""

            warn(:name_missing, "Unnamed #{member[:tagname]}", member)

          end

          (member[:params] || []).each do |p|

          Array(member[:params]).each do |p|

            if !p[:name] || p[:name] == ""

              warn(:name_missing, "Unnamed parameter", member)

            end
@@ -54,7 +54,7 @@ module JsDuck 
      def warn_duplicate_params

        each_member do |member|

          params = {}

          (member[:params] || []).each do |p|

          Array(member[:params]).each do |p|

            if params[p[:name]]

              warn(:dup_param, "Duplicate parameter name #{p[:name]}", member)

            end