Discource-C/discourse
1
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-02-07 20:08:26 +00:00
Code Issues Packages Projects Releases Wiki Activity

PERF: avoid regex for hot path

Browse Source 
Camelize is called quite a lot in zeitwerk, avoid using a regex here which
is far slower than using ends_with?
This commit is contained in:
Sam Saffron 2019-10-08 13:43:54 +11:00
parent 7a5daa3079
commit 586dfcc795
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/freedom_patches/zeitwerk.rb
View File

@ -18,8 +18,8 @@ module ActiveSupport::Dependencies::ZeitwerkIntegration::Inflector
} }
def self.camelize(basename, abspath) def self.camelize(basename, abspath)
return basename.camelize if abspath =~ /onceoff\.rb$/ return basename.camelize if abspath.ends_with?("onceoff.rb")
return 'Jobs' if abspath =~ /jobs\/base\.rb$/ return 'Jobs' if abspath.ends_with?("jobs/base.rb")
CUSTOM_PATHS.fetch(basename, basename.camelize) CUSTOM_PATHS.fetch(basename, basename.camelize)
end end
end end