Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Make dictionary internal table dynamic (and automatically resized).
[simgrid.git] / src / xbt / dict_cursor.c
index 71780eb..4a2986b 100644 (file)
@@ -8,12 +8,13 @@
  * under the terms of the license (GNU LGPL) which comes with this package. */
 
 #include "xbt/misc.h"
+#include "xbt/ex.h"
 #include "dict_private.h"
 
 #include <string.h> /* strlen() */
 
-XBT_LOG_EXTERNAL_CATEGORY(dict);
-XBT_LOG_NEW_DEFAULT_SUBCATEGORY(dict_cursor,dict,"To traverse dictionaries");
+XBT_LOG_EXTERNAL_CATEGORY(xbt_dict);
+XBT_LOG_NEW_DEFAULT_SUBCATEGORY(xbt_dict_cursor,xbt_dict,"To traverse dictionaries");
 
 
 /*####[ Dict cursor functions ]#############################################*/
@@ -21,34 +22,22 @@ XBT_LOG_NEW_DEFAULT_SUBCATEGORY(dict_cursor,dict,"To traverse dictionaries");
 /* Don't add or remove entries to the dict while traversing !!!             */
 /*###########################################################################*/
 struct xbt_dict_cursor_ {
-  /* we store all level encountered until here, to backtrack on next() */
-  xbt_dynar_t   keys;
-  xbt_dynar_t   key_lens;
-  int            pos;
-  int            pos_len;
-  xbt_dictelm_t head;
+  xbt_dictelm_t current;
+  int line;
+  xbt_dict_t dict;
 };
 
-static _XBT_INLINE void
-_cursor_push_keys(xbt_dict_cursor_t p_cursor,
-                  xbt_dictelm_t     p_elm);
-
 #undef xbt_dict_CURSOR_DEBUG
 /*#define xbt_dict_CURSOR_DEBUG 1*/
 
 /** @brief Creator 
- *  @param head   the dict
+ *  @param dict the dict
  */
-xbt_dict_cursor_t 
-xbt_dict_cursor_new(const xbt_dict_t head) {
+xbt_dict_cursor_t xbt_dict_cursor_new(const xbt_dict_t dict) {
   xbt_dict_cursor_t res = NULL;
 
-  res = xbt_new(s_xbt_dict_cursor_t,1);
-  res->keys     = xbt_dynar_new(sizeof(char **), NULL);
-  res->key_lens = xbt_dynar_new(sizeof(int  *),  NULL);
-  res->pos      = 0;
-  res->pos_len  = 0;
-  res->head     = head ? head->head : NULL;
+  res = xbt_new(s_xbt_dict_cursor_t, 1);
+  res->dict = dict;
 
   xbt_dict_cursor_rewind(res);
 
@@ -59,12 +48,9 @@ xbt_dict_cursor_new(const xbt_dict_t head) {
  * @brief Destructor
  * @param cursor poor victim
  */
-void
-xbt_dict_cursor_free(xbt_dict_cursor_t *cursor) {
+void xbt_dict_cursor_free(xbt_dict_cursor_t *cursor) {
   if (*cursor) {
-    xbt_dynar_free(&((*cursor)->keys));
-    xbt_dynar_free(&((*cursor)->key_lens));
-    free(*cursor);
+    xbt_free(*cursor);
     *cursor = NULL;
   }
 }
@@ -72,62 +58,23 @@ xbt_dict_cursor_free(xbt_dict_cursor_t *cursor) {
 /*
  * Sanity check to see if the head contains something
  */
-static _XBT_INLINE
-xbt_error_t
-__cursor_not_null(xbt_dict_cursor_t cursor) {
-
+static XBT_INLINE void __cursor_not_null(xbt_dict_cursor_t cursor) {
   xbt_assert0(cursor, "Null cursor");
-
-  if (!cursor->head) {
-    return mismatch_error;
-  }
-
-  return no_error;
 }
 
 
-static _XBT_INLINE
-void
-_cursor_push_keys(xbt_dict_cursor_t cursor,
-                  xbt_dictelm_t     elm) {
-  xbt_dictelm_t       child = NULL;
-  int                  i       = 0;
-  static volatile int  count   = 0; /* ??? */
-
-  CDEBUG3(dict_cursor, "Push childs of %p (%.*s) in the cursor", (void*)elm, elm->key_len, elm->key);
-
-  if (!elm->internal) {
-    xbt_dynar_push(cursor->keys,     &elm->key    );
-    xbt_dynar_push(cursor->key_lens, &elm->key_len);
-    count++;
-  }
-
-  xbt_dynar_foreach(elm->sub, i, child) {
-    if (child)
-      _cursor_push_keys(cursor, child);
-  }
-
-  CDEBUG1(dict_cursor, "Count = %d", count);
-}
-
 /** @brief Reinitialize the cursor. Mandatory after removal or add in dict. */
-void
-xbt_dict_cursor_rewind(xbt_dict_cursor_t cursor) {
-
-  CDEBUG0(dict_cursor, "xbt_dict_cursor_rewind");
+void xbt_dict_cursor_rewind(xbt_dict_cursor_t cursor) {
+  CDEBUG0(xbt_dict_cursor, "xbt_dict_cursor_rewind");
   xbt_assert(cursor);
 
-  xbt_dynar_reset(cursor->keys);
-  xbt_dynar_reset(cursor->key_lens);
-
-  if (!cursor->head)
-    return ;
-
-  _cursor_push_keys(cursor, cursor->head);
-
-  xbt_dynar_cursor_first(cursor->keys,     &cursor->pos    );
-  xbt_dynar_cursor_first(cursor->key_lens, &cursor->pos_len);
-
+  cursor->line = 0;
+  if (cursor->dict != NULL) {
+    cursor->current = cursor->dict->table[0];
+  }
+  else {
+    cursor->current = NULL;
+  }
 }
 
 /**
@@ -136,30 +83,55 @@ xbt_dict_cursor_rewind(xbt_dict_cursor_t cursor) {
  * @param      dict   on what to let the cursor iterate
  * @param[out] cursor dest address
  */
-void xbt_dict_cursor_first (const xbt_dict_t   dict,
-                            xbt_dict_cursor_t *cursor){
-
+void xbt_dict_cursor_first(const xbt_dict_t   dict,
+                          xbt_dict_cursor_t *cursor){
+  DEBUG0("xbt_dict_cursor_first");
   if (!*cursor) {
     DEBUG0("Create the cursor on first use");
-    *cursor=xbt_dict_cursor_new(dict);
+    *cursor = xbt_dict_cursor_new(dict);
+  }
+  else {
+    xbt_dict_cursor_rewind(*cursor);
+  }
+  if (dict != NULL && (*cursor)->current == NULL) {
+    xbt_dict_cursor_step(*cursor); /* find the first element */
   }
-
-  xbt_dict_cursor_rewind(*cursor);
 }
 
 
 /**
  * \brief Move to the next element. 
  */
-void
-xbt_dict_cursor_step(xbt_dict_cursor_t cursor) {
+void xbt_dict_cursor_step(xbt_dict_cursor_t cursor) {
+
+
+  xbt_dictelm_t current ;
+  int line;
+
+  DEBUG0("xbt_dict_cursor_step");
   xbt_assert(cursor);
 
-  DEBUG2("step cursor. Current=%.*s",
-        xbt_dynar_get_as(cursor->key_lens,cursor->pos_len,int),
-        xbt_dynar_get_as(cursor->keys,cursor->pos,char *));
-  xbt_dynar_cursor_step(cursor->keys,     &cursor->pos);
-  xbt_dynar_cursor_step(cursor->key_lens, &cursor->pos_len);
+  current = cursor->current;
+  line = cursor->line;
+
+  if (cursor->dict != NULL) {
+
+    if (current != NULL) {
+      DEBUG0("current is not null, take the next element");
+      current = current->next;
+      DEBUG1("next element: %p", current);
+    }
+    
+    while (current == NULL && ++line <= cursor->dict->table_size) {
+      DEBUG0("current is NULL, take the next line");
+      current = cursor->dict->table[line];
+      DEBUG1("element in the next line: %p", current);
+    }
+    DEBUG2("search finished, current = %p, line = %d", current, line);
+    
+    cursor->current = current;
+    cursor->line = line;
+  }
 }
 
 /**
@@ -167,74 +139,49 @@ xbt_dict_cursor_step(xbt_dict_cursor_t cursor) {
  *
  * @returns true if it's ok, false if there is no more data
  */
-int
-xbt_dict_cursor_get_or_free(xbt_dict_cursor_t  *cursor,
-                           char               **key,
-                           void               **data) {
-  xbt_error_t  errcode = no_error;
-  int           key_len = 0;
-  
-  if (!cursor || !(*cursor))
-    return FALSE;
+int xbt_dict_cursor_get_or_free(xbt_dict_cursor_t  *cursor,
+                               char               **key,
+                               void               **data) {
 
-  if (xbt_dynar_length((*cursor)->keys) <= (*cursor)->pos) {
-    xbt_dict_cursor_free(cursor);
+  xbt_dictelm_t current;
+
+  DEBUG0("xbt_dict_get_or_free");
+
+
+  if (!cursor || !(*cursor))
     return FALSE;
-  }
-    
-  *key    = xbt_dynar_get_as((*cursor)->keys,     (*cursor)->pos,     char*);
-  key_len = xbt_dynar_get_as((*cursor)->key_lens, (*cursor)->pos_len, int);
 
-  errcode = xbt_dictelm_get_ext((*cursor)->head, *key, key_len, data);
-  if (errcode == mismatch_error) {
+  current = (*cursor)->current;
+  if (current == NULL) { /* no data left */
     xbt_dict_cursor_free(cursor);
     return FALSE;
   }
-
-  xbt_assert1(errcode == no_error,
-              "Unexpected problem while retrieving the content of cursor. Got %s",
-              xbt_error_name(errcode));
-
+  
+  *key = current->key;
+  *data = current->content;
   return TRUE;
 }
 
 /**
  * @brief Get current key
  * @param cursor: the cursor
- * @param key where to put the key
+ * @returns the current key
  */
-xbt_error_t
-xbt_dict_cursor_get_key(xbt_dict_cursor_t   cursor,
-                         /*OUT*/char        **key) {
-  xbt_error_t errcode = no_error;
-
-  TRYOLD(__cursor_not_null(cursor));
-
-  *key = xbt_dynar_get_as(cursor->keys, cursor->pos - 1, char*);
+char *xbt_dict_cursor_get_key(xbt_dict_cursor_t   cursor) {
+  __cursor_not_null(cursor);
 
-  return errcode;
+  return cursor->current->key;
 }
 
 /**
  * @brief Get current data
  * @param cursor the cursor
- * @param data where to put the data
+ * @returns the current data
  */
-xbt_error_t
-xbt_dict_cursor_get_data(xbt_dict_cursor_t   cursor,
-                          /*OUT*/void        **data) {
-  xbt_error_t  errcode = no_error;
-  char         *key     = NULL;
-  int           key_len = 0;
-
-  TRYOLD(__cursor_not_null(cursor));
-
-  key     = xbt_dynar_get_as(cursor->keys,     cursor->pos-1,  char *);
-  key_len = xbt_dynar_get_as(cursor->key_lens, cursor->pos_len-1, int);
-
-  TRYOLD(xbt_dictelm_get_ext(cursor->head, key, key_len, data));
+void *xbt_dict_cursor_get_data(xbt_dict_cursor_t   cursor) {
+  __cursor_not_null(cursor);
 
-  return errcode;
+  return cursor->current->content;
 }