[#104004] [Ruby master Feature#17883] Load bundler/setup earlier to make `bundle exec ruby -r` respect Gemfile — mame@...
Issue #17883 has been reported by mame (Yusuke Endoh).
21 messages
2021/05/24
[ruby-core:103974] [Ruby master Feature#17881] Add a Module#const_added callback
From:
jean.boussier@...
Date:
2021-05-22 11:43:27 UTC
List:
ruby-core #103974
Issue #17881 has been reported by byroot (Jean Boussier). ---------------------------------------- Feature #17881: Add a Module#const_added callback https://bugs.ruby-lang.org/issues/17881 * Author: byroot (Jean Boussier) * Status: Open * Priority: Normal ---------------------------------------- ### Use case Autoloaders like `zeitwerk` need a callback when a new class or module is registered in the constant table. Currently this is implemented with TracePoint's `:class` event. It works, but it is a bit unfortunate to have to use an API intended for debugging to implement production features. It also has performance implications (albeit relatively minor), and [doesn't play well with MJIT](https://k0kubun.medium.com/ruby-3-jit-can-make-rails-faster-756310f235a). ### Proposal I believe that if Ruby was to call `Module#const_added` when a constant is registered, Zeitwerk could get rid of TracePoint. For now I implemented it as: `const_added(const_name)` for similarity with `method_added`. But maybe it could make sense to have the signature be `const_added(const_name, const_value)`. Also since `method_removed` exists, maybe `const_removed` would need to be added for consistency. ### Links Patch: https://github.com/ruby/ruby/pull/4521 Zeitwerk side discussion: https://github.com/fxn/zeitwerk/issues/135 cc @k0kubun -- https://bugs.ruby-lang.org/ Unsubscribe: <mailto:ruby-core-request@ruby-lang.org?subject=unsubscribe> <http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>