Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:78402
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain zend.com designates 209.85.214.173 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <6ADCA194-EB90-4EEB-BCBF-ADD5EAE855AA@ajf.me>
References: <6E04B9BE-854E-4112-8C74-7D90BD8BFE95@ajf.me>
	<CA+9eiLtBKKk4UoJ5VG21qCtqWbmyAK=VmvCdt3+wqgPwwTvDaA@mail.gmail.com>
	<6ADCA194-EB90-4EEB-BCBF-ADD5EAE855AA@ajf.me>
Date: Mon, 27 Oct 2014 23:47:31 +0400
Message-ID: <CA+9eiLvoCxM0uS2N-iwU1ZK-yYHv=0g45Y3LOJTckztCEksOzQ@mail.gmail.com>
To: Andrea Faulds <ajf@ajf.me>
Cc: PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=089e01229ea825f9f405066ccd00
Subject: Re: [PHP-DEV] [RFC] Readonly Properties
From: dmitry@zend.com (Dmitry Stogov)

--089e01229ea825f9f405066ccd00
Content-Type: text/plain; charset=UTF-8

On Mon, Oct 27, 2014 at 10:25 PM, Andrea Faulds <ajf@ajf.me> wrote:

>
> > On 27 Oct 2014, at 18:31, Dmitry Stogov <dmitry@zend.com> wrote:
> >
> > Hi Andrea,
> >
> > I don't have strong opinion about this proposal.
> > It doesn't make any harm to the engine, and it really may speed-up code
> especially written for read-only properties.
> > On the other hand you introduce new orthogonal to
> private/protected/public visibility rule,
> > and I'm not sure if this complication is good for language.
> >
> > Why did you disable read-only static methods? (I just didn't get it).
>
> You couldn't really have a read-only method, there's no get/set equivalent
> for methods. The reason they're explicitly disallowed in the code is
> because, to avoid shift/reduce conflicts, I have to add it as a member
> modifier but then check the AST to see if it was used.
>
> The reason for disallowing static methods is we currently don't use
> separate code paths for get/set of static methods, we just call one
> function to obtain a pointer. Of course I could make that function be told
> whether it's for writing, I just hadn't done it yet.
>

Oh. I meant static properties, of course. (zend_compile.c line 4219)

Thanks. Dmitry.


>
> --
> Andrea Faulds
> http://ajf.me/
>

--089e01229ea825f9f405066ccd00--