Project

General

Profile

Bug #11055

autoload resets private_constant

Added by nobu (Nobuyoshi Nakada) over 4 years ago. Updated about 1 month ago.

Status:
Assigned
Priority:
Normal
Target version:
-
[ruby-core:<unknown>]

Description

Is this behavior intentional?

Suppose the following file:

$ cat a/b.rb
class A; B = 1; end

This causes NameError.

$ ruby -I. -e 'class A;autoload :B, "a/b.rb"; private_constant :B; end' -e 'p A::B'
-e:2:in `<main>': private constant A::B referenced (NameError)

But after it got loaded, A::B becomes public.

$ ruby -I. -e 'class A;autoload :B, "a/b.rb"; private_constant :B; B; end' -e 'p A::B'
1

Files

const-visibility-before-autoload-11055.patch (3.93 KB) const-visibility-before-autoload-11055.patch jeremyevans0 (Jeremy Evans), 08/12/2019 03:59 AM

History

Updated by jeremyevans0 (Jeremy Evans) 3 months ago

I doubt dropping the existing constant visibility information is intentional behavior. You can work around the current behavior by resetting private_constant/deprecate_constant inside the autoloaded file, but that leads to duplication.

Attached is a patch that will copy the constant visibility information across the autoload.

Updated by matz (Yukihiro Matsumoto) about 1 month ago

I agree with fixing this. Nobu will investigate the patch and consider the issue detail.

Matz.

Also available in: Atom PDF