From: "fxn (Xavier Noria)" Date: 2022-03-13T12:52:20+00:00 Subject: [ruby-core:107880] [Ruby master Bug#18622] const_get still looks in Object, while lexical constant lookup no longer does Issue #18622 has been updated by fxn (Xavier Noria). For completeness, let me add that that the described lookup is incomplete. Like all the lookup descriptions I've seen, it obviates `Module#autoload`. Also, the nesting is not strictly lexical. You can push and anonymous module to the nesting of a script with ``ruby load script, true ``` also, you can push an arbitrary class or module to the nesting with ``` A::B::C.eval_family_of_methods(STRING) ``` In any case, my conclusion is that `const_get` does what it is supposed to do. It is not inconsistent. ---------------------------------------- Bug #18622: const_get still looks in Object, while lexical constant lookup no longer does https://bugs.ruby-lang.org/issues/18622#change-96822 * Author: Eregon (Benoit Daloze) * Status: Open * Priority: Normal * ruby -v: ruby 3.0.2p107 (2021-07-07 revision 0db68f0233) [x86_64-linux] * Backport: 2.6: UNKNOWN, 2.7: UNKNOWN, 3.0: UNKNOWN, 3.1: UNKNOWN ---------------------------------------- There is some inconsistency here between literal constant lookup and the meta API (const_get). Lexical constant lookup no longer uses a special case for Object, and this is good as it avoids surprises: #11547 However, `const_get` still looks in Object, even though that's confusing, inconsistent and IMHO shouldn't really happen. ```ruby module ConstantSpecsTwo Foo = :cs_two_foo end module ConstantSpecs end p ConstantSpecs.const_get("ConstantSpecsTwo::Foo") # => :cs_two_foo p ConstantSpecs::ConstantSpecsTwo::Foo # => const_get.rb:9:in `
': uninitialized constant ConstantSpecs::ConstantSpecsTwo (NameError) ``` I think we should change it so both behave the same (i.e., NameError). It's like if `cd /foo/bar` would go to `/bar` if `/foo/bar` does not exist and `/bar` does. `const_get` is a meta API so it cannot know the surrounding `Module.nesting`, but so I think it should consider the receiver of `const_get` as the only nesting (so just `ConstantSpecs` in this case, not `Object`). Note this does not affect nested constants inside the `const_get` like `Foo` above (another way to look at the inconsistency that the first component is treated differently). From https://bugs.ruby-lang.org/issues/11547#note-19 -- https://bugs.ruby-lang.org/ Unsubscribe: