Hi Etienne,
I don't see serious issues in implementation except for I would prefer
to add checks if get_closure handler is NULL.
However in general I don't see any use case for this new handler.
From my point of view, it is always possible to use the Closures class
itself and I cannot imagine case when someone need this handler.
Anyway, I won't object against patch if most of @internals like it.
Thanks. Dmitry.
Etienne Kneuss wrote:
Hi,
http://patches.colder.ch/Zend/zend_get_closure_handler_53.patch?markup
http://patches.colder.ch/Zend/zend_get_closure_handler_HEAD.patch?markup
regards
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
