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/