Steph,

I don't understand your patch. 
(It doesn't compile without my patch and doesn't need with it).

Why you don't just call ts_free_id(*module->globals_id_ptr) and do so large
extra work?

Why you free globals for external extensions and not for internals?

Thanks. Dmitry.


> -----Original Message-----
> From: Steph Fox [mailto:[EMAIL PROTECTED] 
> Sent: Sunday, June 11, 2006 12:49 PM
> To: Stanislav Malyshev
> Cc: Dmitry Stogov; internals@lists.php.net
> Subject: Re: [PHP-DEV] Re: [PATCH] Automatic module globals management
> 
> 
> Hi Stas
> 
> 
> > SF>>resource id, and it would need to be a ZTS build. 
> Whatever way you
> > look at
> > SF>>it, that's a screwed up configuration.
> >
> > So we would blame the user for our broken code. Do you think it's a 
> > good idea.
> 
> No I don't, so I wrote something completely different. Is the 
> attached a 
> better solution?
> Again, it's intended to work _alongside_ Dmitry's, not 
> replace it. It checks 
> for globals_id_ptr, you need to change that to globals_id if 
> you're trying 
> it against current 5_2.
> 
> - Steph
> 
> >
> > --
> > Stanislav Malyshev, Zend Products Engineer
> > [EMAIL PROTECTED]  http://www.zend.com/ +972-3-6139665 ext.115
> >
> 

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

Reply via email to