1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

rb_feature_p: skip get_expanded_load_path for absolute paths

Browse Source 
Ref: https://github.com/fxn/zeitwerk/pull/308

```ruby
require 'benchmark'

$LOAD_PATH << 'relative-path'

autoload :FOO, '/tmp/foo.rb'

puts Benchmark.realtime {
  500_000.times do
    Object.autoload?(:FOO)
  end
}
```

The above script takes 2.5 seconds on `master`, and only
50ms on this branch.

When we're looking for a feature with an absolute path, we don't
need to call the expensive `get_expanded_load_path`.
This commit is contained in:
Jean Boussier 2025-01-12 23:33:00 +01:00
parent dd863714bf
commit d4a1a2780c
Notes: git 2025-01-26 08:52:38 +00:00 
Merged: https://github.com/ruby/ruby/pull/12562
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
load.c
View File

@ -597,7 +597,7 @@ rb_feature_p(rb_vm_t *vm, const char *feature, const char *ext, int rb, int expa
loading_tbl = get_loading_table(vm); loading_tbl = get_loading_table(vm);
f = 0; f = 0;
if (!expanded) { if (!expanded && !rb_is_absolute_path(feature)) {
struct loaded_feature_searching fs; struct loaded_feature_searching fs;
fs.name = feature; fs.name = feature;
fs.len = len; fs.len = len;