> It was clearly and unambiguously a patch. It wasn't in diff > format, but it > was clearly indicating that to make your code more portable, you could > change the line: > > HE *hes [count]; > to > #if defined(__BORLANDC__) || defined(_MSC_VER) || > defined(__STDC__) > HE **hes = Perl_malloc (count * sizeof (HE)); > #else > HE *hes [count]; > #endif 1) an overlook - it should be HE **hes = Perl_malloc (count * sizeof (HE*)); it appears that HE is larger than HE* and allocating more memory, hence misbehaviour is not noticed, but anyway, this isn't correct 2) when that chunk of memory freed? TIA :)Thread Previous | Thread Next