Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:79469
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain zend.com designates 209.85.220.177 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAFp73Xvj6_hokca4LMciNrUZ2rtF50UbZjYCsyZuagRqXzC-hw@mail.gmail.com>
References: <CAFp73Xtmw6PVxN0XaKkjJC5M5PU-W=prpYDob5aXHuQzE5dwNw@mail.gmail.com>
	<CAFMT4Npr-urz0hDHmn1E9JniURgjFD8fOKUGFf1zgpYR-+_O9w@mail.gmail.com>
	<CAFp73Xugd2novgpwGDruWU-p2K=8GZwKQ1mus=c6C6nZ1EzwqA@mail.gmail.com>
	<CAFp73Xvj6_hokca4LMciNrUZ2rtF50UbZjYCsyZuagRqXzC-hw@mail.gmail.com>
Date: Mon, 8 Dec 2014 14:45:00 +0400
Message-ID: <CA+9eiLvf4uAdOgZdjAtij30Ugpy8U6mcPEosRAAdPOiy_ySN4g@mail.gmail.com>
To: "guilhermeblanco@gmail.com" <guilhermeblanco@gmail.com>, Nikita Popov <nikita.ppv@gmail.com>
Cc: Levi Morrison <levim@php.net>, PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a11c2b5a8509d770509b21e58
Subject: Re: [PHP-DEV] Zend language parser improvements
From: dmitry@zend.com (Dmitry Stogov)

--001a11c2b5a8509d770509b21e58
Content-Type: text/plain; charset=UTF-8

I don't see technical problems with the patch.
However, I also don't see any significant benefits.
From the user perspective it'll just change error messages and prevent
"final final class" declarations.

Nikita, what do you think?

Thanks. Dmitry.

On Fri, Dec 5, 2014 at 8:18 PM, guilhermeblanco@gmail.com <
guilhermeblanco@gmail.com> wrote:

> Hi guys,
>
> I'd really appreciate some review around the before-mentioned PRs. I have
> added a new one to the list now:
>
> - https://github.com/php/php-src/pull/937
>
> This PR addresses the parsing support for traits to have "extends" and
> "implements", as they are invalid.
>
> There's another one in the oven, which prevents extension developers to
> create classes that extends traits or interfaces.
> This is currently supported only for userland classes, but not for Zend
> API.
>
> Cheers,
>
> On Wed, Dec 3, 2014 at 8:39 PM, guilhermeblanco@gmail.com <
> guilhermeblanco@gmail.com> wrote:
>
> >
> >
> > On Wed, Dec 3, 2014 at 8:06 PM, Levi Morrison <levim@php.net> wrote:
> >
> >> The parser changes need to be careful reviewed; I don't have time at
> >> the moment to verify it but I think you unintentionally allowed some
> >> syntax's that shouldn't be valid because of the addition to
> >> `inner_statement`.
> >>
> >
> > Shouldn't. I broke down class_declaration_statement into 3 pieces:
> > class_declaration_statement, interface_declaration_statement and
> > trait_declaration_statement.
> > At the end, all I've done is adding the other 2 new rules back to where
> it
> > was consumed.
> >
> >
> >>
> >> Maybe I just looked too quickly. In any case, parser changes should
> >> always get several people reviewing them.
> >>
> >
> > Agreed. =)
> >
> > --
> > Guilherme Blanco
> > MSN: guilhermeblanco@hotmail.com
> > GTalk: guilhermeblanco
> > Toronto - ON/Canada
> >
>
>
>
> --
> Guilherme Blanco
> MSN: guilhermeblanco@hotmail.com
> GTalk: guilhermeblanco
> Toronto - ON/Canada
>

--001a11c2b5a8509d770509b21e58--