rails / sprockets

Rack-based asset packaging system
MIT License
949 stars 789 forks source link

Performance sucker! -> Get performance doubled! #503

Closed janbiedermann closed 6 years ago

janbiedermann commented 7 years ago

sprockets-3.7.1/lib/sprockets/uri_utils.rb in def split_file_uri(uri)

Line 51

      # Hack for parsing Windows "file:///C:/Users/IEUser" paths
      path.gsub!(/^\/([a-zA-Z]:)/, '\1'.freeze)

is a performance sucker. Replacing it with this:

      # Hack for parsing Windows "file:///C:/Users/IEUser" paths
      path.gsub!(/^\/([a-zA-Z]:)/, '\1'.freeze) if path[9].include?(':')

DOUBLES performance for me, that is injavascript_include_tag 'applications.js', generating the Githubissues.

  • Githubissues is a development platform for aggregating issues.