Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:64071
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: unknown (pb1.pair.com: domain zend.com does not designate 209.85.216.49 as permitted sender)
MIME-Version: 1.0
Date: Wed, 28 Nov 2012 13:57:58 +0300
Message-ID: <CA+9eiLtSMLdeye=HQBx4koT3FfWaY5-a_v7mZiOT_i4jqgzGsQ@mail.gmail.com>
To: "nikita.ppv" <nikita.ppv@gmail.com>, Xinchen Hui <laruence@gmail.com>, derek <derek@zend.com>, 
	Yoram Bar Haim <yoram.b@zend.com>, PHP Internals <internals@lists.php.net>, Zeev Suraski <zeev@zend.com>, 
	Andi Gutmans <andi@zend.com>
Content-Type: multipart/alternative; boundary=047d7b6789641e860c04cf8c0ae1
Subject: Generators Revisited
From: dmitry@zend.com (Dmitry Stogov)

--047d7b6789641e860c04cf8c0ae1
Content-Type: text/plain; charset=UTF-8

Hi

I've tried to review the generators related changes made in PHP-5.5 and
improve them from both consistency and performance points of view.
The patch also makes a small performance improvement to PHP-5.5 in general,
however, it changes some ZE internals that can be used by
some extension (e.g. XDebug).

You can find the list of changes in the UPGRADING file.

http://pastebin.com/AT0SQhAH

Please, review the patch and test it with your custom extensions.

I would like to commit it on Friday morning. Objections?

I also have some additional proposals:

1) EX(Ts) access optimization. With this patch it must be possible to
access temporary variables using "execute_data" as the base pointer.
"execute_data.Ts" might be removed at all. In general the same optimization
might be done for EX(CVs) as well.

2) May be it makes sense to forbid return/yield usage in "finally" blocks
using compile time error. Their behaviour is really unclear.

Any thoughts?

Thanks. Dmitry.

--047d7b6789641e860c04cf8c0ae1--