Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:56684
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain googlemail.com designates 209.85.215.42 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <4ED6713D.2050009@ralphschindler.com>
References: <4ED6713D.2050009@ralphschindler.com>
Date: Wed, 30 Nov 2011 19:51:46 +0100
Message-ID: <CAF+90c8vKhvOO3qTPgN2TsV-=Vsw0G14B=-HQbD1YZw-Wt6bmg@mail.gmail.com>
To: Ralph Schindler <ralph@ralphschindler.com>
Cc: internals <internals@lists.php.net>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] 5.4's New De-referencing plus assignment
From: nikita.ppv@googlemail.com (Nikita Popov)

Hi Ralph!

This (in it's current form) doesn't look like a good idea to me. You
are only handling yet another edge case. ($foo =3D $bar =3D new A)->bar()
would again not work.

If we really want this kind of (expr)-> syntax, then it should really
accept *any* expr. But I am not quite sure whether this will work out
easily because in this case we can't be sue that expr is an object
(but maybe it'll be simple, I just don't know).

Nikita

On Wed, Nov 30, 2011 at 7:09 PM, Ralph Schindler
<ralph@ralphschindler.com> wrote:
> Hey all (Filipe),
>
> This was brought up in the thread "New dereferencing syntaxes in 5.4".
>
> I too think this would be beneficial:
>
> =A0$value =3D ($obj =3D new Foo)->produceAValue();
>
> but the current parser (branch 5.4) doesn't have a rule for this. =A0I've
> attached a quick/working patch, but I don't fully understand the
> ramifications of such a patch.
>
> Can someone explain the drawbacks of increasing the "%expect n" for me?
>
> Would this many s/r require/benefit from a new expression grouping?
>
> More to the point, can we have this small change for 5.4?
>
> Thanks in advance.
> -ralph
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php