X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=lib%2Fshash.c;h=4285c07451fba0b29b16efb90293d5928e134bca;hb=1d4fd3a5e7aa37624f04f02b42b4e7c075cd138c;hp=3269a4108c38474dd703f34580a7fd2ec749255a;hpb=c56d226f8d3fc7a68716be87e3d651be604213fa;p=sliver-openvswitch.git diff --git a/lib/shash.c b/lib/shash.c index 3269a4108..4285c0745 100644 --- a/lib/shash.c +++ b/lib/shash.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2009, 2010 Nicira Networks. + * Copyright (c) 2009, 2010, 2011, 2012 Nicira, Inc. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -16,11 +16,11 @@ #include #include "shash.h" -#include #include "hash.h" static struct shash_node *shash_find__(const struct shash *, - const char *name, size_t hash); + const char *name, size_t name_len, + size_t hash); static size_t hash_name(const char *name) @@ -108,7 +108,7 @@ shash_add_nocopy__(struct shash *sh, char *name, const void *data, size_t hash) { struct shash_node *node = xmalloc(sizeof *node); node->name = name; - node->data = (void *) data; + node->data = CONST_CAST(void *, data); hmap_insert(&sh->map, &node->node, hash); return node; } @@ -144,7 +144,7 @@ void shash_add_assert(struct shash *sh, const char *name, const void *data) { bool added OVS_UNUSED = shash_add_once(sh, name, data); - assert(added); + ovs_assert(added); } /* Searches for 'name' in 'sh'. If it does not already exist, adds it along @@ -156,13 +156,13 @@ shash_replace(struct shash *sh, const char *name, const void *data) size_t hash = hash_name(name); struct shash_node *node; - node = shash_find__(sh, name, hash); + node = shash_find__(sh, name, strlen(name), hash); if (!node) { shash_add_nocopy__(sh, xstrdup(name), data, hash); return NULL; } else { void *old_data = node->data; - node->data = (void *) data; + node->data = CONST_CAST(void *, data); return old_data; } } @@ -189,12 +189,13 @@ shash_steal(struct shash *sh, struct shash_node *node) } static struct shash_node * -shash_find__(const struct shash *sh, const char *name, size_t hash) +shash_find__(const struct shash *sh, const char *name, size_t name_len, + size_t hash) { struct shash_node *node; HMAP_FOR_EACH_WITH_HASH (node, node, hash, &sh->map) { - if (!strcmp(node->name, name)) { + if (!strncmp(node->name, name, name_len) && !node->name[name_len]) { return node; } } @@ -205,7 +206,15 @@ shash_find__(const struct shash *sh, const char *name, size_t hash) struct shash_node * shash_find(const struct shash *sh, const char *name) { - return shash_find__(sh, name, hash_name(name)); + return shash_find__(sh, name, strlen(name), hash_name(name)); +} + +/* Finds and returns a shash_node within 'sh' that has the given 'name' that is + * exactly 'len' bytes long. Returns NULL if no node in 'sh' has that name. */ +struct shash_node * +shash_find_len(const struct shash *sh, const char *name, size_t len) +{ + return shash_find__(sh, name, len, hash_bytes(name, len, 0)); } void * @@ -232,7 +241,7 @@ void * shash_find_and_delete_assert(struct shash *sh, const char *name) { void *data = shash_find_and_delete(sh, name); - assert(data != NULL); + ovs_assert(data != NULL); return data; } @@ -267,7 +276,7 @@ shash_sort(const struct shash *sh) SHASH_FOR_EACH (node, sh) { nodes[i++] = node; } - assert(i == n); + ovs_assert(i == n); qsort(nodes, n, sizeof *nodes, compare_nodes_by_name); @@ -303,57 +312,3 @@ shash_random_node(struct shash *sh) { return CONTAINER_OF(hmap_random_node(&sh->map), struct shash_node, node); } - -/* String-to-string maps (smaps). */ - -/* Frees 'smap', including its keys and string values. */ -void -smap_destroy(struct shash *smap) -{ - shash_destroy_free_data(smap); -} - -/* Returns true if string-to-string maps 'a' and 'b' contain the same keys and - * values, false otherwise. */ -bool -smap_equal(const struct shash *a, const struct shash *b) -{ - struct shash_node *a_node; - - if (shash_count(a) != shash_count(b)) { - return false; - } - - SHASH_FOR_EACH (a_node, a) { - uint32_t hash = a_node->node.hash; - struct shash_node *b_node = shash_find__(b, a_node->name, hash); - if (!b_node || strcmp(a_node->data, b_node->data)) { - return false; - } - } - - return true; -} - -/* Initializes 'dst' as a clone of 'src'. */ -void -smap_clone(struct shash *dst, const struct shash *src) -{ - struct shash_node *node; - - shash_init(dst); - SHASH_FOR_EACH (node, src) { - shash_add_nocopy__(dst, xstrdup(node->name), xstrdup(node->data), - node->node.hash); - } -} - -/* Adds 'key' with string 'value' to 'smap', making a copy of each. - * - * It is the caller's responsibility to avoid duplicate names, if that is - * desirable. */ -void -smap_add(struct shash *smap, const char *key, const char *value) -{ - shash_add(smap, key, xstrdup(value)); -}