Feature #15777: autoload?(cname, inherit=true)

• Status: Closed
• Priority: Normal
• Assignee: nobu (Nobuyoshi Nakada)
• Target version: -

[fxn (Xavier Noria)]

Zeitwerk needs to be able to check if a given class or module has an autoload defined for a certain constant name, that would be autoload?(cname, false), similar to const_defined?(cname, false).

[fxn (Xavier Noria)]

• Description updated (diff)

[byroot (Jean Boussier)]

I made a patch for this feature: https://github.com/ruby/ruby/pull/2173

[nobu (Nobuyoshi Nakada)]

Changing the signature of rb_autoload_p could break API compatibility.
I think we'll need another function for it.
Maybe rb_autoload_at_p like as rb_const_defined_at?

[byroot (Jean Boussier)]

nobu (Nobuyoshi Nakada) I edited the patch the way you suggested. Let me know if you desire further changes.

[nobu (Nobuyoshi Nakada)]

Is it necessary to export rb_autoload_at_p?
Do you have the plan to use it in extension libraries?

[nobu (Nobuyoshi Nakada)]

byroot (Jean Boussier) wrote:

Let me know if you desire further changes.

Fix the bug first.
In rb_autoload_at_p, this condition is wrong.

-    while (!autoload_defined_p(mod, id)) {
+    while (RTEST(recur) && autoload_defined_p(mod, id)) {

Leave it unchanged but return nil if not recur in that loop.

[byroot (Jean Boussier)]

Is it necessary to export rb_autoload_at_p?
Do you have the plan to use it in extension libraries?

No I have no such plan. I did so because it's defined in variable.c but called from load.c so without that declaration it won't compile:

load.c:1180:12: error: implicit declaration of function 'rb_autoload_at_p' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
    return rb_autoload_at_p(mod, id, recur);

I'm quite a MRI & C beginner though, so if there is a better way to do it I'll gladly change it.

Eregon (Benoit Daloze) told me which place to declare internal functions. It's now in internal.h

return nil if not recur in that loop.

Done.

[Eregon (Benoit Daloze)]

• Assignee set to nobu (Nobuyoshi Nakada)

The new patch looks fine to me.
nobu (Nobuyoshi Nakada) I'll let you merge it since you did most of the review.

[byroot (Jean Boussier)]

• Status changed from Open to Closed

Applied in changeset git|fb85a428605265a8fd449b0702a4dd88cb6f3b20.

---

Add an optional inherit argument to Module#autoload?

[Feature #15777]

Closes: https://github.com/ruby/ruby/pull/2173