diff options
author | Evan Huus <eapache@gmail.com> | 2013-06-27 03:01:53 +0000 |
---|---|---|
committer | Evan Huus <eapache@gmail.com> | 2013-06-27 03:01:53 +0000 |
commit | ffe6d9c4d6fe36d7f2498154cdb12159b6b60aff (patch) | |
tree | 8434e17f9ca7dd947fb50b92d7a52fde3074a0ec | |
parent | 32f401052783b9e41799e10cd44acf602b877b75 (diff) |
Misc little cleanups to wmem tree (comments, whitespace, assertions)
svn path=/trunk/; revision=50182
-rw-r--r-- | epan/wmem/wmem_tree.c | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/epan/wmem/wmem_tree.c b/epan/wmem/wmem_tree.c index 2d380419e6..001fc5fe1d 100644 --- a/epan/wmem/wmem_tree.c +++ b/epan/wmem/wmem_tree.c @@ -307,7 +307,7 @@ lookup_or_insert32(wmem_tree_t *tree, guint32 key, { wmem_tree_node_t *node = tree->root; wmem_tree_node_t *new_node = NULL; - + /* is this the first node ?*/ if (!node) { new_node = create_node(tree->allocator, NULL, key, @@ -344,7 +344,7 @@ lookup_or_insert32(wmem_tree_t *tree, guint32 key, node = node->right; } else { - /* new node to the left */ + /* new node to the right */ new_node = create_node(tree->allocator, node, key, CREATE_DATA(func, data), WMEM_NODE_COLOR_RED, is_subtree); @@ -552,9 +552,7 @@ wmem_tree_insert32_array(wmem_tree_t *tree, wmem_tree_key_t *key, void *data) guint32 i, insert_key32 = 0; for (cur_key = key; cur_key->length > 0; cur_key++) { - if (cur_key->length > 100) { - g_assert_not_reached(); - } + g_assert(cur_key->length < 100); for (i = 0; i < cur_key->length; i++) { /* Insert using the previous key32 */ @@ -568,10 +566,7 @@ wmem_tree_insert32_array(wmem_tree_t *tree, wmem_tree_key_t *key, void *data) } } - if (!insert_tree) { - /* We didn't get a valid key. Should we return NULL instead? */ - g_assert_not_reached(); - } + g_assert(insert_tree); wmem_tree_insert32(insert_tree, insert_key32, data); } |