
27 Apr
2011
27 Apr
'11
3:39 p.m.
On Tue, Apr 12, 2011 at 13:10:15 -0600, Eric Blake wrote:
On 04/12/2011 11:25 AM, Jiri Denemark wrote:
So far first entries for each hash key are stored directly in the hash table while other entries mapped to the same key are linked through pointers. As a result of that, the code is cluttered with special handling for the first items.
This patch makes all entries (even the first ones) linked through pointers, which significantly simplifies the code and makes it more maintainable. --- src/util/hash.c | 294 +++++++++++++++++-------------------------------------- 1 files changed, 92 insertions(+), 202 deletions(-)
ACK - all the changes look correct, and certainly simpler.
Thanks, I pushed the patch since we have the unit test now. Jirka