aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEvan Huus <eapache@gmail.com>2013-06-27 03:01:53 +0000
committerEvan Huus <eapache@gmail.com>2013-06-27 03:01:53 +0000
commitffe6d9c4d6fe36d7f2498154cdb12159b6b60aff (patch)
tree8434e17f9ca7dd947fb50b92d7a52fde3074a0ec
parent32f401052783b9e41799e10cd44acf602b877b75 (diff)
Misc little cleanups to wmem tree (comments, whitespace, assertions)
svn path=/trunk/; revision=50182
-rw-r--r--epan/wmem/wmem_tree.c13
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);
}