Changeset 55b77d9 in mainline for kernel/generic/src/adt/hash_table.c

Timestamp:

2011-06-17T20:39:16Z (13 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

8f164724

Parents:

98caf49

Message:

Separate list_t typedef from link_t (kernel part).

• list_t represents lists
• Use list_first(), list_last(), list_empty() where appropriate
• Use list_foreach() where possible
• Replace improper uses of list_prepend() with list_insert_after()
• Replace improper uses of list_append() with list_insert_before()

File:

• kernel/generic/src/adt/hash_table.c (modified) (6 diffs)

kernel/generic/src/adt/hash_table.c

@@ -62 +62 @@
         ASSERT(max_keys > 0);
         
-        h->entry = (link_t *) malloc(m * sizeof(link_t), 0);
+        h->entry = (list_t *) malloc(m * sizeof(list_t), 0);
         if (!h->entry)
                 panic("Cannot allocate memory for hash table.");
         
-        memsetb(h->entry, m * sizeof(link_t), 0);
+        memsetb(h->entry, m * sizeof(list_t), 0);
         
         for (i = 0; i < m; i++)
@@ -107 +107 @@
 link_t *hash_table_find(hash_table_t *h, sysarg_t key[])
 {
-        link_t *cur;
         size_t chain;
         
@@ -118 +117 @@
         ASSERT(chain < h->entries);
         
-        for (cur = h->entry[chain].next; cur != &h->entry[chain]; cur = cur->next) {
+        list_foreach(h->entry[chain], cur) {
                 if (h->op->compare(key, h->max_keys, cur)) {
                         /*
@@ -141 +140 @@
 {
         size_t chain;
-        link_t *cur;
         
         ASSERT(h);
@@ -150 +148 @@
         
         if (keys == h->max_keys) {
+                link_t *cur;
         
                 /*
@@ -169 +168 @@
          */
         for (chain = 0; chain < h->entries; chain++) {
-                for (cur = h->entry[chain].next; cur != &h->entry[chain]; cur = cur->next) {
+                list_foreach(h->entry[chain], cur) {
                         if (h->op->compare(key, keys, cur)) {
                                 link_t *hlp;