Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:88076
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain siobud.com designates 104.236.58.159 as permitted sender)
Date: Mon, 7 Sep 2015 01:43:10 -0500
To: Andrea Faulds <ajf@ajf.me>
Cc: internals@lists.php.net
Message-ID: <20150907064310.GA12497@3006.local>
References: <20150827173432.GA71000@3006.local>
 <28.A5.59944.15C7CE55@pb1.pair.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <28.A5.59944.15C7CE55@pb1.pair.com>
User-Agent: Mutt/1.5.23 (2014-03-12)
Subject: Re: [PHP-DEV] Re: [RFC] [Concept] Class Constant visibility
 modifiers in PHP 7.1+
From: sean@siobud.com (Sean DuBois)

On Sun, Sep 06, 2015 at 06:47:56PM +0100, Andrea Faulds wrote:
> Hi Sean,
>
> Sean DuBois wrote:
> >
> >I am starting this discussion to get peoples opinion on the overall feature, and find someone
> >who would be interested in watching over my progress and making sure I
> >do the right things to hopefully get this merged.
> >
> >The PHP bug tracker contains a few simple entries for a adding visibility modifiers to class constants.
> >https://bugs.php.net/bug.php?id=27022
> >
> >I would be the one implementing this, and have a basic working version already (that
> >takes shortcuts like reusing property_info) but it works!
> >https://github.com/Sean-Der/php-src/compare/master...bug-69980-class-constants#diff-6231c13c8582758f41a5e2a015e3b5c5R1
> >There are cases where runtime/compile time checks pass/fail incorrectly,
> >but working on that now.
> >
> >This change would involve breaking the API, but wouldn't involved any
> >language breaking changes. All current const declarations would just
> >default to public and keep the same behavior.
>
> Funny you should propose this now, I was wanting this feature just the other
> day. This would be a useful addition, and clean up a strange inconsistency:
> why can methods and properties have visibility modifiers, but not constants?
>
> I'd love to see this in PHP. Are you going to write an RFC?
>
> Thanks!
> --
> Andrea Faulds
> http://ajf.me/
>
Just one of those things that you you wish you had, but then end up
solving with other things (static properties) so we will see how long it
takes me to blunder through it :)

Reeze Xia started an RFC, but didn't have time to finish it. We talked
over GitHub and I will be taking it over fully.
https://wiki.php.net/rfc/class_const_visibility

I am just waiting on RFC Karma, after I get that I will start filling it in!