Re: something about neo_hash

Brandon Long <[email protected]> Wed, 3 Jun 2009 01:31:47 -0700
Newsgroups gmane.text.clearsilver.general
Organization Fiction L Networks
Message-ID <20090603083147.GB17670@bl1>
Can you provide test cases for whatever bug you think these fix?

Brandon

On 06/03/09 [email protected] uttered the following other thing:
> Hi,
> 
> This is something confused me in neo_hash mode:
> 
> 1, dose neo_hash.c line 146 should change to 
> 
> node = _hash_lookup_node(hash, *key, NULL);
> 
> 
> 2, and before we judge if (*node) in line 163, should we judge if (node) first ? Because the *key parameter can be null, and so node is null.
> 
> 
> 3, it seems that 
> 
> #define hash_insert ne_hash_insert
> 
> is missing in ClearSilver.h
> 
> 
> 
> ------------------------------------
> 
> Yahoo! Groups Links
> 
> 
> 
-- 
 "Its much more fun to be sand than oil in the machinery of life."
                                           http://www.fiction.net/blong/