[ruby-core:34967] Re: [Ruby 1.9-Feature#3908][Assigned] private constant

Hi, matz

May I introduce these methods in relation to private constants?

  - Module#public_constants, as alias to Module#constants
    (corresponding to Ojbect#public_methods)

  - Module#private_constants, which returns an array that contains
    only private constant names.
    (corresponding to Ojbect#private_methods)

  - Module#public_const_get, which return the constant value or
    raise a NameError if the constant is private
    (corresponding to Object#public_send)

  - Module#public_const_defined?, which tells whether the public
    constant is defined or not
    (corresponding to Module#public_method_defined?)

  - Module#private_const_defined?, which tells whether the private
    constant is defined or not
    (corresponding to Module#private_method_defined?)

A patch and tests are attached.  Thank you.



Note: I fixed other bugs which RPRR and nagachika reported (r30713,
r30714, r30715, r30716 and r30718).  Sorry for late action.


2010/12/22 Yusuke Endoh <[e-mail]>:
> Issue #3908 has been updated by Yusuke Endoh.
>
> Status changed from Closed to Assigned
>
> Hi, NaHi
>
>> Just as a notification since I see this ticket closed today.
>
> Kazu committed a change of NEWS with ML ref, which closed this
> ticket.  think that he did not intend to close this ticket.
> That is, it is just accident. 'm reopening this ticket.
>
>
>> Yusuke, patches in the mail I'm replying are not applied
>
> Yes, just because of my laziness.  will work on.
>
>
>> and we still need discuss about followings, right?
>
> I think that what we needed is not discussion, but matz's final
> approval.
> The reason why this ticket stopped was because matz cancelled
> his approval and opposed this feature at [ruby-dev:42469].
> And, matz has finally re-approved it at [ruby-dev:42587].
> So we can now move on.
>
>
>> - Module#constants includes a private constant?
>
> It should not.  will.
>
>
>> - Module#const_get raises NameError for a private constant?
>
> It should not.  will.
>
>
>> - add Module#public_constants and Module#private_constants?
>
> Plus, Module#public_const_get. ut these may be need matz's
> additional approval.
>
>
> Matz, can I add these methods:
>
>  Module#public_constants, which returns an array that contains
> nly public constant names.
>
>  Module#private_constants, which returns an array that contatins
> nly private constant names.
>
>  Module#public_const_get, which returns a value assigned to the
> onstant if it is public, or raise a NameError if it is private.
>
> ?
>
>
>> - Module#const_set changes the constant visibility to public? Setting
> visibility via const_set?
>
> It is just a bug.  will.
>
> --
> Yusuke Endoh <[e-mail]>
> ----------------------------------------
> http://redmine.ruby-lang.org/issues/show/3908
>
> ----------------------------------------
> http://redmine.ruby-lang.org


-- 
Yusuke Endoh <[e-mail]>