Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:91644
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.213.171 as permitted sender)
Content-Type: text/plain; charset=us-ascii
Mime-Version: 1.0 (Mac OS X Mail 9.2 \(3112\))
In-Reply-To: <CAESVnVrm9_cxy_dJfmoYVEeJtoj0qVdA2d54A4x7KPJfFcbGmQ@mail.gmail.com>
Date: Sat, 12 Mar 2016 11:54:26 -0600
Cc: Nikita Popov <nikita.ppv@gmail.com>,
 PHP internals <internals@lists.php.net>
Content-Transfer-Encoding: quoted-printable
Message-ID: <321D2AE8-4C59-4739-9D8B-C3A55D5B7D8C@gmail.com>
References: <CAF+90c-EMQ41jmrdCKytMpB7hXQUEvQ8biCaZTgAEOw-4WpcQA@mail.gmail.com> <CAESVnVrm9_cxy_dJfmoYVEeJtoj0qVdA2d54A4x7KPJfFcbGmQ@mail.gmail.com>
To: Sara Golemon <pollita@php.net>
Subject: Re: [PHP-DEV] Evaluate arguments of new for classes without constructors
From: pmjones88@gmail.com ("Paul M. Jones")


> On Mar 12, 2016, at 11:27, Sara Golemon <pollita@php.net> wrote:
>=20
>> HHVM does not implement "new" in this way, they always evaluate the =
arguments.
>>=20
> Which, if it helps, means that we already know a lot of frameworks
> /don't/ break as a result of fixing this behavior.
>=20
>> As this is technically a BC affecting change (even if of the lowest
>> impact), I'm running it past the list first.
>>=20
> Technically a BC break that justifies waiting till a major version.
> Given the extremely specific requirements to hit this edge case
> however, passing side-effect args to a non-existent constructor, I
> personally think a minor version is fine.

Of curiosity, what effect (if any) might this have on =
ReflectionClass::newInstance(), ::newInstanceArgs(), and =
::newInstanceWithoutConstructor()?  (I ask because some existing DI =
systems, such as Aura.Di, rely on those for creating objects.)


--=20
Paul M. Jones
pmjones88@gmail.com
http://paul-m-jones.com

Modernizing Legacy Applications in PHP
https://leanpub.com/mlaphp

Solving the N+1 Problem in PHP
https://leanpub.com/sn1php