Re: glib 1.2 patch.



On Fri, 9 Apr 1999, Miguel de Icaza wrote:
> > is g_hash_table_insert (). g_hash_table_insert () sets
> > the key and value fields. This just seems like a waste
> > of (tiny amounts of) CPU cycles to me. 

> Morten uses Purify extensively to check our applications.  
> 
> Since the node is never actually released but put on the free list, as
> far as purify is concerned the data kept on the hash table is valid,
> and makes harder to debug leaks (ie, you might leak but purify wont
> catch this since there is still a reference to it).

Seems to me like the patch would be acceptable if the code is enabled
only for debug versions.

	Jeff







[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]