On 2003/12/20, at 0:21, Dmitry Stogov wrote:
Hi Moriyoshi,
Yes I am sure. Andi approved this patch today.
I didn't know about your patch. May be you know some situations when my patch will not work properly?
As far as I looked over the patch, your patch appears the right fix for the problem. I asked you it because I was told by Andi that the garbage collection is sensitive stuff and handled with extreme care. Actually I think nothing will be wrong with it. However, I suspect that we don't need to replicate the same garbage collecting code to that place. I think all we have to do is delay the GC timing so it would be performed after the error handler is executed.
Just to explain in a bit more detail, your atomic postponing of garbage collecting isn't good because too much can accumulate and/or the life time of garbage could be too long.
I think Dmitry's patch handles this slightly in a different way and better although as I said, a thorough fix should be done.
Andi
-- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
