Project

General

Profile

Actions

Bug #11055

closed

autoload resets private_constant

Added by nobu (Nobuyoshi Nakada) almost 9 years ago. Updated over 4 years ago.

Status:
Closed
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

Updated by jeremyevans0 (Jeremy Evans) over 4 years 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) over 4 years ago

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

Matz.

Actions #3

Updated by jeremyevans (Jeremy Evans) over 4 years ago

  • Status changed from Assigned to Closed

Applied in changeset git|47c97e1e843159c3c4d57f8c5e22daea57c3ffe1.


Do not lose existing constant visibility when autoloading

This copies the private/deprecate constant visibility across the
autoload. It still is backwards compatible with setting the
private/deprecate constant visibility in the autoloaded file.
However, if you explicitly set public constant in the autoloaded
file, that will be reset after the autoload.

Fixes [Bug #11055]

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0