Re: [ZEND-ENGINE-CVS] cvs: Zend / zend_execute.c zend_execute.h
| From: | Thies C. Arntzen | Date: | Tue, 01 Oct 2002 16:07:46 +0000 |
| Subject: | Re: [ZEND-ENGINE-CVS] cvs: Zend / zend_execute.c zend_execute.h | ||
| References: | 1 2 3 | Groups: | php.zend-engine.cvs |
| Request: | Send a blank email to php-zend-engine-cvs+get-461@lists.php.net to get a copy of this message | ||
On Tue, Oct 01, 2002 at 06:31:49PM +0300, Zeev Suraski wrote:
> At 18:19 01/10/2002, Derick Rethans wrote:
> >On Tue, 1 Oct 2002, Zeev Suraski wrote:
> >
> >> zeev Tue Oct 1 11:03:04 2002 EDT
> >>
> >> Modified files:
> >> /Zend zend_execute.c zend_execute.h
> >> Log:
> >> execute_internal() shouldn't be exposed. Thies - if you think it
> >SHOULD be
> >> exposed for some reason, then it should move to zend_execute_API.c
> >
> >It should be exposed so that extensions can hook into this. You just
> >broke some extensions here :)
>
> I thought you'd hook through zend_execute_internal, and not
> execute_internal. Anyway, if you really do need execute_internal, then we
> can move it to zend_execute_API.c.
you do hook into zend_execute_internal, but you still need to
be able to perform call the "real-thing", which is
execute_internal in this case. i modelled it after the
zend_execute and execute naming you choose in the 1st place.
re,
tc
>
> Zeev
--
Thies C. Arntzen - Looking for all sorts of freelance work - just ask..
Whishlist: http://www.amazon.de/exec/obidos/wishlist/AB9DY62QWDSZ