From: Ben Pfaff <blp@nicira.com>
Date: Tue, 28 Jul 2009 20:29:57 +0000 (-0700)
Subject: shash: Make shash_add() return the new node.
X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=78299b0a6871a31c8b0cce5ef3f67ab465ab094f;p=openvswitch

shash: Make shash_add() return the new node.
---

diff --git a/lib/shash.c b/lib/shash.c
index 93163727..f1785139 100644
--- a/lib/shash.c
+++ b/lib/shash.c
@@ -57,15 +57,16 @@ shash_is_empty(const struct shash *shash)
     return hmap_is_empty(&shash->map);
 }
 
-/* It is the caller's responsible to avoid duplicate names, if that is
+/* It is the caller's responsibility to avoid duplicate names, if that is
  * desirable. */
-void
+struct shash_node *
 shash_add(struct shash *sh, const char *name, void *data)
 {
     struct shash_node *node = xmalloc(sizeof *node);
     node->name = xstrdup(name);
     node->data = data;
     hmap_insert(&sh->map, &node->node, hash_name(name));
+    return node;
 }
 
 void
diff --git a/lib/shash.h b/lib/shash.h
index bcac41b5..72d83020 100644
--- a/lib/shash.h
+++ b/lib/shash.h
@@ -35,7 +35,7 @@ void shash_init(struct shash *);
 void shash_destroy(struct shash *);
 void shash_clear(struct shash *);
 bool shash_is_empty(const struct shash *);
-void shash_add(struct shash *, const char *, void *);
+struct shash_node *shash_add(struct shash *, const char *, void *);
 void shash_delete(struct shash *, struct shash_node *);
 struct shash_node *shash_find(const struct shash *, const char *);
 void *shash_find_data(const struct shash *, const char *);