It looks like extensions that don't use this new API will continue to
work, they'll just leak when __set() is used with them.

We should bump the zend API number to allow extensions to make use of
the new API.

--Wez.

On 3/29/06, Pierre <[EMAIL PROTECTED]> wrote:
> On Wed, 29 Mar 2006 14:28:43 -0000
> [EMAIL PROTECTED] ("Antony Dovgal") wrote:
>
>
> >   fix bug #36898 (__set() leaks in classes extending internal ones)
> >
> >   Added:
> >   ZEND_API void zend_object_std_init(zend_object *object,
> > zend_class_entry *ce TSRMLS_DC) ZEND_API void
> > zend_object_std_dtor(zend_object *object TSRMLS_DC)
> >   to initialize and destroy zend_object structs
>
> Without minimizing the importance of this fix, I wonder two things. The
> first is about adding such things right when we are in the RC phase for
> 5.1.3, it sounds a bit risky to me.
>
> The second problem is about extension available in pecl as well, all of
> a sudden, it is not possible to compile them with the last php 5.1
> version, and there is no possible way now to compile them.
>
> What's about a Zend api version change? and then allows an extension to
> be compiled with previous PHP versions.
>
> Regards,
>
> --Pierre
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to