Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:65110
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.223.177 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAEPJdun2fpS6zh-SdhgNot24Nw=i7_aNaPxxhQybMq4Gsm-pQA@mail.gmail.com>
References: <50F840F4.7080704@zerocue.com>
	<50FE7579.1010409@zerocue.com>
	<50FECA4E.6080408@lerdorf.com>
	<CAEPJdun2fpS6zh-SdhgNot24Nw=i7_aNaPxxhQybMq4Gsm-pQA@mail.gmail.com>
Date: Wed, 23 Jan 2013 12:35:57 +0100
Message-ID: <CAH-PCH4HxWJcgMw63c_g2Lqy3wmO2rCsk0r7EmQJPCHf3N+3vA@mail.gmail.com>
To: Benjamin Eberlei <kontakt@beberlei.de>
Cc: Rasmus Lerdorf <rasmus@lerdorf.com>, Clint Priest <cpriest@zerocue.com>, 
	PHP Developers Mailing List <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=14dae934066507b85304d3f31957
Subject: Re: [PHP-DEV] [VOTE] Property Accessors for 5.5
From: tyra3l@gmail.com (Ferenc Kovacs)

--14dae934066507b85304d3f31957
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

On Wed, Jan 23, 2013 at 12:11 PM, Benjamin Eberlei <kontakt@beberlei.de>wro=
te:

> On Tue, Jan 22, 2013 at 6:20 PM, Rasmus Lerdorf <rasmus@lerdorf.com>
> wrote:
>
> > On 01/22/2013 03:18 AM, Clint Priest wrote:
> > >
> > > On 1/17/2013 12:20 PM, Clint Priest wrote:
> > >> I'm happy to say that Property Accessors is ready for a vote for
> > >> inclusion in 5.5 release.
> > >>
> > >> Nikita and I (as well as Stas a bit) have all been working hard to
> > >> make this happen for 5.5, voting and the specifications are here:
> > >>
> > >> https://wiki.php.net/rfc/propertygetsetsyntax-v1.2#voting
> > >>
> > >
> > > For those that have voted against this proposal, are there any
> > > clarifications that can be made or questions answered?
> > >
> > > There seems to be a lot of userland support for this proposal from
> > > people who don't have voting rights.
> >
> > The simple explanation from me is that the ROI isn't there on this one.
> > It adds a lot of code complexity for very little return. Yes, it saves =
a
> > couple of lines of boilerplate code for a few people, but the cost is
> > high in terms of ongoing maintenance and potential issues for opcode
> > caches as well. If you look at the split in voting you will notice it i=
s
> > pretty much split along the lines of the people who have to maintain
> > this code vs. the people who would like a shiny new feature.
> >
>
> I think it would be important to gather the reasons for the No Votes and
> ways how to address this problems.
> Some from the previous discusisons are:
>
> * Neglected APC support
> * adding complexity to PHP Core
> * Reference Handling
> * Visibility Rules
> * Fear of testing in production
>
> And come up with solutions to those (like cutting some features, or putti=
ng
> the code into 5.6 to allow for more testing)
>
> Since several contributors helped with the development and committed
> themselves for quite some time already, then if there were a chance to
> address all the concerns then this feature could still move forward (mayb=
e
> at a later point)?
>
>
Hi,

for the record I'm one of the people who thinks that this is a good idea
and the implementation improved a bunch from the previous
versions/proposals, but I think that there were too many changes which
could have introduced new bugs (both in design and implementation).
Accepting this feature just before the feature freeze would either prolong
the 5.5 release or we could end up in the same spot as with 5.4, where
major problems with the new features were discovered after the release plus
some of the must-have extensions wasn't ready to work with the new version.
I don't wanna hijack the discussion, but I think there are a couple of
concerns raised here would worth a separate discussion and possibly changes
in our rfc and release process.

--=20
Ferenc Kov=C3=A1cs
@Tyr43l - http://tyrael.hu

--14dae934066507b85304d3f31957--