Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:83106
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain zend.com designates 209.85.160.181 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAF+90c-+zdywtuzVvpqxY_ZmrsAH++z5_kHMo9Lg1k_dJfYnMQ@mail.gmail.com>
References: <CAEbg-4aZytKiqG3Y5U-HrBMp2+JuGHu3UTOMLWiiUQn59FtpLw@mail.gmail.com>
	<CA+9eiLvkoTB2xdo4a6AdDnU8fjxOBbog+4FHm-UyBZzWNOQd4Q@mail.gmail.com>
	<CAF+90c8tSa640KrkzMk0kCE8x06m=y007u4zyB9_Zrh7vP3mJg@mail.gmail.com>
	<CAEbg-4ZoLQ_MKPz4b9PoCuLj5rBYBOnSbnF4c2PG4odV=Vv=Rg@mail.gmail.com>
	<CAF+90c-+zdywtuzVvpqxY_ZmrsAH++z5_kHMo9Lg1k_dJfYnMQ@mail.gmail.com>
Date: Wed, 18 Feb 2015 23:22:13 +0400
Message-ID: <CA+9eiLvVySGcxaB5q4a-tyUaKQE5wnv+asJni+s++BZXO5u+pg@mail.gmail.com>
To: Nikita Popov <nikita.ppv@gmail.com>
Cc: Alexander Lisachenko <lisachenko.it@gmail.com>, Nikita Popov <nikic@php.net>, 
	PHP internals list <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a1133e5f29868a4050f61bce5
Subject: Re: [PHP-DEV] [RFC][Discussion] Parser extension API
From: dmitry@zend.com (Dmitry Stogov)

--001a1133e5f29868a4050f61bce5
Content-Type: text/plain; charset=UTF-8

I think the AST API shouldn't use "public" properties.
Using it, we will have to construct the whole tree of objects, duplicating
information from AST.
I would propose SimpleXML approach instead - construct object only for
node(s) we currently access.

So at first we will construct just single object referring to AST root.
Then traversing it we will create and destroy objects for necessary nodes.

To access children I would propose to implementing ArrayAccess interface.


$ast = \php\ast\parse($string);
foreach ($ast as $child) {
   echo "\t" . $child->getKindName() . "\n";
   foreach ($child as $grandchild) {
   echo "\t" . $child->getKindName() . "\n";

   }
}

Thanks. Dmitry.

On Wed, Feb 18, 2015 at 10:06 PM, Nikita Popov <nikita.ppv@gmail.com> wrote:

> On Wed, Feb 18, 2015 at 4:22 PM, Alexander Lisachenko <
> lisachenko.it@gmail.com> wrote:
>
>>
>> 2015-02-18 17:59 GMT+03:00 Nikita Popov <nikita.ppv@gmail.com>:
>>
>>> Alexander, I would recommend you to split this into two RFCs, one
>>> dealing only with AST export (and maybe pretty printing) and the second one
>>> with the compilation hooks. There's probably a few questions about the
>>> export API that should be discussed that will be forgotten if everyone
>>> focuses on the more complicated, and probably not yet relevant, question of
>>> compilation hooks.
>>
>>
>>
>> Hello, Nikita! Thanks for you thoughts!
>>
>> Sounds reasonable for me, because of short timeframe for PHP7. Let's go
>> with the first part of RFC, because second part is really cumbersome and
>> can be applied later, eg. in 7.1 I will split this RFC into two parts and
>> mark first part for 7.0.
>>
>
> Great!
>
> About first part, do you agree with proposed classes and namespace
>> `Php\Parser`?
>>
>
> Yeah, I'm okay with that namespace. I prefer php\ast as being more
> distinctive from existing projects, but that's not really important.
>
>
>> I want to propose to use classes for parser API instead of pure functions
>> because it can give more usable OO-API for future needs.
>>
>
> I'm okay with having stuff like ->getKindName() on the nodes, however I'd
> still keep around the free-standing functions, because they can be used
> without a node (i.e. you only need the kind AST_FOO and not an instantiated
> node).
>
> I also don't see why we need ParserEngine::parse() instead of just a
> (namespaced) parse() function. Not a fan of unnecessary wrapper classes.
>
>
>> There is one more notice regarding to the zval nodes, they should be also
>> exported as objects, not as values. I think we should export coherent nodes
>> for everything.
>>
>
> What's the advantage of this? Most leaf nodes will be zval nodes and I
> feel it would be rather inconvenient if they'd all have extra wrappers. Is
> there some specific problem with including the values directly?
>
> Nikita
>
>

--001a1133e5f29868a4050f61bce5--