Re: Re: cvs: php-src(PHP_5_1) / NEWS /ext/com_dotnet com_persist.c /ext/date php_date.c /ext/dom php_dom.c /ext/mysqli mysqli.c /ext/reflection
php_reflection.c /ext/simplexml simplexml.c /ext/spl spl_array.c spl_directory.c spl_iterators.c
| From: | Wez Furlong | Date: | Wed, 29 Mar 2006 19:54:28 +0000 |
| Subject: | Re: Re: cvs: php-src(PHP_5_1) / NEWS /ext/com_dotnet com_persist.c /ext/date php_date.c /ext/dom php_dom.c /ext/mysqli mysqli.c /ext/reflection php_reflection.c /ext/simplexml simplexml.c /ext/spl spl_array.c spl_directory.c spl_iterators.c |
||
| Groups: | php.internals | ||
| Request: | Send a blank email to internals+get-22615@lists.php.net to get a copy of this message | ||
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 <pierre.php@gmail.com> wrote:
> On Wed, 29 Mar 2006 14:28:43 -0000
> tony2001@php.net ("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
>
>