Re: internal hashtables [patch included]

At 02:53 PM 10/26/2003 -0200, Cristiano Duarte wrote:

Hi all, Just to remind about my problem: After updating from latest CVS I still had problems with internal hashtables. I want to make internal class constants and the engine reports memory leaks. So I made this patch wich works for me and removed all memory leaks regarding hashtables. This patch introduces a new macro called ZEND_INIT_INTERNAL_SYMTABLE wich is to be used instead of ZEND_INIT_SYMTABLE when the hashtables is internal. That's the only change that should be done in sources that needs internal hashtables.

This doesn't look right. Shouldn't you make sure that module shutdown frees the hashtables? You can change the dtor if you need. I'm sorry but I don't quite understand what made you come up with this solution. Maybe I'm missing something. Andi