Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Some symbols were renamed by last flexml update.
[simgrid.git] / src / xbt / graph.c
index 02e32bc..50eee15 100644 (file)
@@ -1,13 +1,13 @@
-/*     $Id$     */
-
 /* a generic graph library.                                                 */
 
-/* Copyright (c) 2006 Darina Dimitrova, Arnaud Legrand.                     */
-/* All rights reserved.                                                     */
+/* Copyright (c) 2006, 2007, 2008, 2009, 2010. The SimGrid Team.
+ * All rights reserved.                                                     */
 
 /* This program is free software; you can redistribute it and/or modify it
  * under the terms of the license (GNU LGPL) which comes with this package. */
 
+#include "simgrid_config.h" /* getline */
+#include <errno.h>
 #include <stdlib.h>
 #include "xbt/sysdep.h"
 #include "xbt/log.h"
 #include "xbt/graphxml_parse.h"
 #include "xbt/dict.h"
 #include "xbt/heap.h"
+#include "xbt/str.h"
 
 
 
 
-XBT_LOG_NEW_DEFAULT_SUBCATEGORY(graph, xbt, "Graph");
+XBT_LOG_NEW_DEFAULT_SUBCATEGORY(xbt_graph, xbt, "Graph");
 
 
 
-/** Constructor
- * \return a new graph
+/** @brief Constructor
+ *  @return a new graph
  */
 xbt_graph_t xbt_graph_new_graph(unsigned short int directed, void *data)
 {
@@ -39,12 +40,16 @@ xbt_graph_t xbt_graph_new_graph(unsigned short int directed, void *data)
   return graph;
 }
 
+/** @brief add a node to the given graph */
 xbt_node_t xbt_graph_new_node(xbt_graph_t g, void *data)
 {
   xbt_node_t node = NULL;
   node = xbt_new0(struct xbt_node, 1);
   node->data = data;
-  node->in = xbt_dynar_new(sizeof(xbt_edge_t), NULL);
+  if (g->directed)
+    /* only the "out" field is used */
+    node->in = xbt_dynar_new(sizeof(xbt_edge_t), NULL);
+
   node->out = xbt_dynar_new(sizeof(xbt_edge_t), NULL);
   node->position_x = -1.0;
   node->position_y = -1.0;
@@ -54,286 +59,328 @@ xbt_node_t xbt_graph_new_node(xbt_graph_t g, void *data)
   return node;
 }
 
-
+/** @brief add an edge to the given graph */
 xbt_edge_t xbt_graph_new_edge(xbt_graph_t g,
-                             xbt_node_t src, xbt_node_t dst, void *data)
+                              xbt_node_t src, xbt_node_t dst, void *data)
 {
   xbt_edge_t edge = NULL;
 
   edge = xbt_new0(struct xbt_edge, 1);
   xbt_dynar_push(src->out, &edge);
-  if (g->directed) 
+  if (g->directed)
     xbt_dynar_push(dst->in, &edge);
-  else /* only the "out" field is used */
+  else                          /* only the "out" field is used */
     xbt_dynar_push(dst->out, &edge);
 
   edge->data = data;
   edge->src = src;
   edge->dst = dst;
-  if (!g->directed) 
-    {
-      xbt_dynar_push(src->in, &edge);
-      xbt_dynar_push(dst->out, &edge);
-    }
-
 
   xbt_dynar_push(g->edges, &edge);
 
   return edge;
 }
 
+/** @brief Get the edge connecting src and dst */
+xbt_edge_t xbt_graph_get_edge(xbt_graph_t g, xbt_node_t src,
+                              xbt_node_t dst)
+{
+  xbt_edge_t edge;
+  unsigned int cursor;
 
-/** Destructor
- * \param l poor victim
- *
- * Free the graph structure. 
- */
-void xbt_graph_free_graph(xbt_graph_t g,
-                         void node_free_function(void *ptr),
-                         void edge_free_function(void *ptr),
-                         void graph_free_function(void *ptr))
+  xbt_dynar_foreach(src->out, cursor, edge) {
+    XBT_DEBUG("%p = %p--%p", edge, edge->src, edge->dst);
+    if ((edge->src == src) && (edge->dst == dst))
+      return edge;
+  }
+  if (!g->directed) {
+    xbt_dynar_foreach(src->out, cursor, edge) {
+      XBT_DEBUG("%p = %p--%p", edge, edge->src, edge->dst);
+      if ((edge->dst == src) && (edge->src == dst))
+        return edge;
+    }
+  }
+  return NULL;
+}
+
+/** @brief Get the user data associated to a node */
+void *xbt_graph_node_get_data(xbt_node_t node)
 {
-  int cursor = 0;
-  xbt_node_t node = NULL;
-  xbt_edge_t edge = NULL;
+  return node->data;
+}
 
+/** @brief Set the user data associated to a node */
+void xbt_graph_node_set_data(xbt_node_t node, void *data)
+{
+  node->data = data;
+}
 
-  xbt_dynar_foreach(g->nodes, cursor, node)
-    {
-      xbt_dynar_free(&(node->out));
-      xbt_dynar_free(&(node->in));
-      if(node_free_function)
-       node_free_function(node->data);
-    }
+/** @brief Get the user data associated to a edge */
+void *xbt_graph_edge_get_data(xbt_edge_t edge)
+{
+  return edge->data;
+}
 
-  xbt_dynar_foreach(g->edges, cursor, edge) 
-    {
-      if(edge_free_function)
-       edge_free_function(edge->data);
-    }
+/** @brief Set the user data associated to a edge */
+void xbt_graph_edge_set_data(xbt_edge_t edge, void *data)
+{
+  edge->data = data;
+}
 
-  xbt_dynar_foreach(g->nodes, cursor, node)
-    free(node);
-  xbt_dynar_free(&(g->nodes));
+/** @brief Destructor
+ *  @param g: poor victim
+ *  @param node_free_function: function to use to free data associated to each node
+ *  @param edge_free_function: function to use to free data associated to each edge
+ *  @param graph_free_function: function to use to free data associated to g
+ *
+ * Free the graph structure.
+ */
+void xbt_graph_free_graph(xbt_graph_t g,
+                          void_f_pvoid_t node_free_function,
+                          void_f_pvoid_t edge_free_function,
+                          void_f_pvoid_t graph_free_function)
+{
+  unsigned int cursor;
+  xbt_node_t node;
+  xbt_edge_t edge;
 
-  xbt_dynar_foreach(g->edges, cursor, edge)
+  xbt_dynar_foreach(g->edges, cursor, edge) {
+    if (edge_free_function)
+      edge_free_function(edge->data);
     free(edge);
+  }
   xbt_dynar_free(&(g->edges));
 
-  free(g);
+  xbt_dynar_foreach(g->nodes, cursor, node) {
+    xbt_dynar_free(&(node->out));
+    xbt_dynar_free(&(node->in));
+    if (node_free_function)
+      node_free_function(node->data);
+    free(node);
+  }
+  xbt_dynar_free(&(g->nodes));
 
-  return;
+  if (graph_free_function)
+    graph_free_function(g->data);
+  free(g);
+  xbt_graph_parse_lex_destroy();
 }
 
 
-
+/** @brief remove the given node from the given graph */
 void xbt_graph_free_node(xbt_graph_t g, xbt_node_t n,
-                        void_f_pvoid_t * node_free_function,
-                        void_f_pvoid_t * edge_free_function)
+                         void_f_pvoid_t node_free_function,
+                         void_f_pvoid_t edge_free_function)
 {
   unsigned long nbr;
-  int i;
-  int cursor = 0;
+  unsigned long i;
+  unsigned int cursor = 0;
   xbt_node_t node = NULL;
   xbt_edge_t edge = NULL;
 
   nbr = xbt_dynar_length(g->edges);
-  cursor=0;
-  for (i = 0; i < nbr; i++)
-    {
-      xbt_dynar_cursor_get(g->edges, &cursor, &edge);
-        
-      if ((edge->dst == n) || (edge->src == n))
-       {
-         xbt_graph_free_edge(g, edge, edge_free_function);
-       } 
-      else xbt_dynar_cursor_step( g->edges, &cursor);   
-    }
+  cursor = 0;
+  for (i = 0; i < nbr; i++) {
+    xbt_dynar_get_cpy(g->edges, cursor, &edge);
 
+    if ((edge->dst == n) || (edge->src == n)) {
+      xbt_graph_free_edge(g, edge, edge_free_function);
+    } else
+      cursor++;
+  }
 
- if ((node_free_function) && (n->data))
 if ((node_free_function) && (n->data))
     node_free_function(n->data);
 
   cursor = 0;
   xbt_dynar_foreach(g->nodes, cursor, node)
-    {
       if (node == n)
-       xbt_dynar_cursor_rm(g->nodes, &cursor);
+    xbt_dynar_cursor_rm(g->nodes, &cursor);
 
-    }
+  xbt_dynar_free(&(n->in));
+  xbt_dynar_free(&(n->out));
+
+  free(n);
 
   return;
 }
 
+/** @brief remove the given edge from the given graph */
 void xbt_graph_free_edge(xbt_graph_t g, xbt_edge_t e,
-                        void free_function(void *ptr))
+                         void_f_pvoid_t free_function)
 {
   int idx;
-  int cursor = 0;
-  xbt_edge_t edge = NULL; 
+  unsigned int cursor = 0;
+  xbt_edge_t edge = NULL;
 
   if ((free_function) && (e->data))
     free_function(e->data);
 
-  xbt_dynar_foreach(g->edges, cursor, edge) 
-    {
-    if (edge == e)
-      {
-       if (g->directed) {
-         idx = __xbt_find_in_dynar(edge->dst->in,edge); 
-         xbt_dynar_remove_at(edge->dst->in, idx,NULL);
-       } else { /* only the out field is used */
-         idx = __xbt_find_in_dynar(edge->dst->out,edge); 
-         xbt_dynar_remove_at(edge->dst->out, idx,NULL);
-       }
-
-       idx = __xbt_find_in_dynar(edge->src->out,edge);
-       xbt_dynar_remove_at(edge->src->out,idx,NULL);   
-
-        xbt_dynar_cursor_rm(g->edges, &cursor); 
-       free(edge);
-       break;
+  xbt_dynar_foreach(g->edges, cursor, edge) {
+    if (edge == e) {
+      if (g->directed) {
+        idx = __xbt_find_in_dynar(edge->dst->in, edge);
+        xbt_dynar_remove_at(edge->dst->in, idx, NULL);
+      } else {                  /* only the out field is used */
+        idx = __xbt_find_in_dynar(edge->dst->out, edge);
+        xbt_dynar_remove_at(edge->dst->out, idx, NULL);
       }
+
+      idx = __xbt_find_in_dynar(edge->src->out, edge);
+      xbt_dynar_remove_at(edge->src->out, idx, NULL);
+
+      xbt_dynar_cursor_rm(g->edges, &cursor);
+      free(edge);
+      break;
     }
+  }
 }
 
 int __xbt_find_in_dynar(xbt_dynar_t dynar, void *p)
 {
 
-  int cursor = 0;
-  void *tmp=NULL;
+  unsigned int cursor = 0;
+  void *tmp = NULL;
 
-  xbt_dynar_foreach(dynar, cursor, tmp)
-    {
-      if (tmp == p)
-       return cursor;
-    }
+  xbt_dynar_foreach(dynar, cursor, tmp) {
+    if (tmp == p)
+      return cursor;
+  }
   return -1;
 }
 
+/** @brief Retrieve the graph's nodes as a dynar */
 xbt_dynar_t xbt_graph_get_nodes(xbt_graph_t g)
 {
   return g->nodes;
 }
 
+/** @brief Retrieve the graph's edges as a dynar */
 xbt_dynar_t xbt_graph_get_edges(xbt_graph_t g)
 {
   return g->edges;
 }
 
+/** @brief Retrieve the node at the source of the given edge */
 xbt_node_t xbt_graph_edge_get_source(xbt_edge_t e)
 {
-
   return e->src;
 }
 
+/** @brief Retrieve the node being the target of the given edge */
 xbt_node_t xbt_graph_edge_get_target(xbt_edge_t e)
 {
   return e->dst;
 }
 
+/** @brief Retrieve the outgoing edges of the given node */
+xbt_dynar_t xbt_graph_node_get_outedges(xbt_node_t n)
+{
+  return n->out;
+}
 
+/** @brief Set the weight of the given edge */
 void xbt_graph_edge_set_length(xbt_edge_t e, double length)
 {
   e->length = length;
 
 }
 
-double xbt_graph_edge_get_length(xbt_edge_t e)
+/** @brief Get the length of a edge */
+double xbt_graph_edge_get_length(xbt_edge_t edge)
 {
-  return e->length;
+  return edge->length;
 }
 
 
-/*construct the adjacency matrix corresponding to a graph,
-  the weights are the distances between nodes
+/** @brief construct the adjacency matrix corresponding to the given graph
+ *
+ * The weights are the distances between nodes
  */
 double *xbt_graph_get_length_matrix(xbt_graph_t g)
 {
-  int cursor = 0;
-  int in_cursor = 0;
-  int  idx,i;
+  unsigned int cursor = 0;
+  unsigned int in_cursor = 0;
+  unsigned long idx, i;
   unsigned long n;
   xbt_edge_t edge = NULL;
-  xbt_node_t node=NULL;
+  xbt_node_t node = NULL;
   double *d = NULL;
 
 # define D(u,v) d[(u)*n+(v)]
   n = xbt_dynar_length(g->nodes);
 
-  d = (double *) xbt_new0(double, n*n);
+  d = (double *) xbt_new0(double, n * n);
 
-  for (i = 0; i < n * n; i++)
-    {
-      d[i] = -1.0;
-    }
-  xbt_dynar_foreach(g->nodes, cursor, node) 
-    {
-      in_cursor = 0;
-      D(cursor, cursor) = 0;
-    
-      xbt_dynar_foreach(node->out, in_cursor, edge)
-       {
-         if (edge->dst==node) 
-           idx= __xbt_find_in_dynar(g->nodes, edge->src);
-         else /*case of  undirected graphs*/ 
-            idx = __xbt_find_in_dynar(g->nodes, edge->dst);
-         D( cursor,idx) = edge->length;
-       }
+  for (i = 0; i < n * n; i++) {
+    d[i] = -1.0;
+  }
+
+  xbt_dynar_foreach(g->nodes, cursor, node) {
+    in_cursor = 0;
+    D(cursor, cursor) = 0;
+
+    xbt_dynar_foreach(node->out, in_cursor, edge) {
+      if (edge->dst == node)
+        idx = __xbt_find_in_dynar(g->nodes, edge->src);
+      else                      /*case of  undirected graphs */
+        idx = __xbt_find_in_dynar(g->nodes, edge->dst);
+      D(cursor, idx) = edge->length;
     }
+  }
 
 # undef D
 
   return d;
 }
 
-
+/** @brief Floyd-Warshall algorithm for shortest path finding
+ *
+ * From wikipedia:
+ *
+ * The Floyd–Warshall algorithm takes as input an adjacency matrix
+ * representation of a weighted, directed graph (V, E). The weight of a
+ * path between two vertices is the sum of the weights of the edges along
+ * that path. The edges E of the graph may have negative weights, but the
+ * graph must not have any negative weight cycles. The algorithm computes,
+ * for each pair of vertices, the minimum weight among all paths between
+ * the two vertices. The running time complexity is Θ(|V|3).
+ */
 void xbt_floyd_algorithm(xbt_graph_t g, double *adj, double *d,
-                        xbt_node_t * p)
+                         xbt_node_t * p)
 {
-  int i, j, k;
+  unsigned long i, j, k;
   unsigned long n;
   n = xbt_dynar_length(g->nodes);
 
 # define D(u,v) d[(u)*n+(v)]
 # define P(u,v) p[(u)*n+(v)]
 
-  for (i = 0; i < n * n; i++)
-    {
-      d[i] = adj[i];
-    }
+  for (i = 0; i < n * n; i++) {
+    d[i] = adj[i];
+  }
 
 
-  for (i = 0; i < n; i++)
-    {
-      for (j = 0; j < n; j++)
-       {
-         if (D(i, j) != -1)
-           {
-             P(i,j) =*((xbt_node_t*) xbt_dynar_get_ptr(g->nodes, i));       
-           }
-       }
+  for (i = 0; i < n; i++) {
+    for (j = 0; j < n; j++) {
+      if (D(i, j) != -1) {
+        P(i, j) = *((xbt_node_t *) xbt_dynar_get_ptr(g->nodes, i));
+      }
     }
-  for (k = 0; k < n; k++)
-    {
-      for (i = 0; i < n; i++)
-       {
-         for (j = 0; j < n; j++)
-           {
-             if ((D(i, k) != -1) && (D(k, j) != -1))
-               {
-                 if ((D(i, j) == -1) || (D(i, j) > D(i, k) + D(k, j)))
-                   {
-                     D(i, j) = D(i, k) + D(k, j);
-                     P(i, j) = P(k, j);
-                   }
-               }
-           }
-       }
+  }
+
+  for (k = 0; k < n; k++) {
+    for (i = 0; i < n; i++) {
+      for (j = 0; j < n; j++) {
+        if ((D(i, k) != -1) && (D(k, j) != -1)) {
+          if ((D(i, j) == -1) || (D(i, j) > D(i, k) + D(k, j))) {
+            D(i, j) = D(i, k) + D(k, j);
+            P(i, j) = P(k, j);
+          }
+        }
+      }
     }
+  }
 
 
 
@@ -341,12 +388,12 @@ void xbt_floyd_algorithm(xbt_graph_t g, double *adj, double *d,
 # undef D
 }
 
-/*computes all-pairs shortest paths*/
+/** @brief computes all-pairs shortest paths */
 xbt_node_t *xbt_graph_shortest_paths(xbt_graph_t g)
 {
   xbt_node_t *p;
   xbt_node_t *r;
-  int i, j, k;
+  unsigned long i, j, k;
   unsigned long n;
 
   double *adj = NULL;
@@ -357,29 +404,25 @@ xbt_node_t *xbt_graph_shortest_paths(xbt_graph_t g)
 
   n = xbt_dynar_length(g->nodes);
   adj = xbt_graph_get_length_matrix(g);
-  d = xbt_new0(double,n*n);
-  p = xbt_new0(xbt_node_t,n*n);
-  r = xbt_new0(xbt_node_t,n*n);
+  d = xbt_new0(double, n * n);
+  p = xbt_new0(xbt_node_t, n * n);
+  r = xbt_new0(xbt_node_t, n * n);
+
   xbt_floyd_algorithm(g, adj, d, p);
-  for (i = 0; i < n; i++)
-    {
-      for (j = 0; j < n; j++)
-       {
-         k = j;
-
-         while ((P(i, k)) && (__xbt_find_in_dynar(g->nodes, P(i, k)) != i))
-           {
-             k = __xbt_find_in_dynar(g->nodes, P(i, k));
-           }
-
-         if (P(i, j))
-           {
-             R(i, j) = *((xbt_node_t*) xbt_dynar_get_ptr(g->nodes, k));
-           }
-       }
+
+  for (i = 0; i < n; i++) {
+    for (j = 0; j < n; j++) {
+      k = j;
+
+      while ((P(i, k)) && (__xbt_find_in_dynar(g->nodes, P(i, k)) != i)) {
+        k = __xbt_find_in_dynar(g->nodes, P(i, k));
+      }
+
+      if (P(i, j)) {
+        R(i, j) = *((xbt_node_t *) xbt_dynar_get_ptr(g->nodes, k));
+      }
     }
+  }
 # undef R
 # undef P
 
@@ -389,116 +432,187 @@ xbt_node_t *xbt_graph_shortest_paths(xbt_graph_t g)
   return r;
 }
 
-
-xbt_edge_txbt_graph_spanning_tree_prim(xbt_graph_t g)
+/** @brief Extract a spanning tree of the given graph */
+xbt_edge_t *xbt_graph_spanning_tree_prim(xbt_graph_t g)
 {
-  int tree_size=0;
-  int tree_size_max=xbt_dynar_length(g->nodes)-1;
-  xbt_edge_t *tree = xbt_new0(xbt_edge_t,tree_size_max);
-  xbt_edge_t e,edge;
+  int tree_size = 0;
+  int tree_size_max = xbt_dynar_length(g->nodes) - 1;
+  xbt_edge_t *tree = xbt_new0(xbt_edge_t, tree_size_max);
+  xbt_edge_t e, edge;
   xbt_node_t node = NULL;
   xbt_dynar_t edge_list = NULL;
-  xbt_heap_t heap = xbt_heap_new(10,NULL);
-  int cursor;
+  xbt_heap_t heap = xbt_heap_new(10, NULL);
+  unsigned int cursor;
 
-  xbt_assert0(!(g->directed),
-             "Spanning trees do not make sense on directed graphs");
+  xbt_assert(!(g->directed),
+              "Spanning trees do not make sense on directed graphs");
 
   xbt_dynar_foreach(g->nodes, cursor, node) {
     node->xbtdata = NULL;
   }
 
-  node = xbt_dynar_getfirst_as(g->nodes,xbt_node_t);
-  node->xbtdata = (void*) 1;
+  node = xbt_dynar_getfirst_as(g->nodes, xbt_node_t);
+  node->xbtdata = (void *) 1;
   edge_list = node->out;
   xbt_dynar_foreach(edge_list, cursor, e)
-    xbt_heap_push(heap,e, -(e->length));
-  
-  while((edge=xbt_heap_pop(heap))) {
-    if((edge->src->xbtdata) && (edge->dst->xbtdata)) continue;
-    tree[tree_size++]=edge;
-    if(!(edge->src->xbtdata)) {
-      edge->src->xbtdata = (void*) 1;
+      xbt_heap_push(heap, e, -(e->length));
+
+  while ((edge = xbt_heap_pop(heap))) {
+    if ((edge->src->xbtdata) && (edge->dst->xbtdata))
+      continue;
+    tree[tree_size++] = edge;
+    if (!(edge->src->xbtdata)) {
+      edge->src->xbtdata = (void *) 1;
       edge_list = edge->src->out;
       xbt_dynar_foreach(edge_list, cursor, e) {
-       xbt_heap_push(heap,e, -(e->length));
+        xbt_heap_push(heap, e, -(e->length));
       }
     } else {
-      edge->dst->xbtdata = (void*) 1;
+      edge->dst->xbtdata = (void *) 1;
       edge_list = edge->dst->out;
       xbt_dynar_foreach(edge_list, cursor, e) {
-       xbt_heap_push(heap,e, -(e->length));
+        xbt_heap_push(heap, e, -(e->length));
       }
     }
-    if(tree_size==tree_size_max) break;
+    if (tree_size == tree_size_max)
+      break;
   }
-  
+
   xbt_heap_free(heap);
 
   return tree;
 }
 
+/** @brief Topological sort on the given graph
+ *
+ *  From wikipedia:
+ *
+ * In graph theory, a topological sort of a directed acyclic graph (DAG) is
+ * a linear ordering of its nodes which is compatible with the partial
+ * order R induced on the nodes where x comes before y (xRy) if there's a
+ * directed path from x to y in the DAG. An equivalent definition is that
+ * each node comes before all nodes to which it has edges. Every DAG has at
+ * least one topological sort, and may have many.
+ */
+xbt_node_t *xbt_graph_topo_sort(xbt_graph_t g)
+{
+
+  xbt_node_t *sorted;
+  unsigned int cursor;
+  int idx;
+  xbt_node_t node;
+  unsigned long n;
+
+  n = xbt_dynar_length(g->nodes);
+  idx = n - 1;
+
+  sorted = xbt_malloc(n * sizeof(xbt_node_t));
+
+  xbt_dynar_foreach(g->nodes, cursor, node)
+      node->xbtdata = xbt_new0(int, 1);
+
+  xbt_dynar_foreach(g->nodes, cursor, node)
+      xbt_graph_depth_visit(g, node, sorted, &idx);
+
+  xbt_dynar_foreach(g->nodes, cursor, node) {
+    free(node->xbtdata);
+    node->xbtdata = NULL;
+  }
+
+  return sorted;
+}
+
+/** @brief First-depth graph traversal */
+void xbt_graph_depth_visit(xbt_graph_t g, xbt_node_t n,
+                           xbt_node_t * sorted, int *idx)
+{
+  unsigned int cursor;
+  xbt_edge_t edge;
+
+  if (*((int *) (n->xbtdata)) == ALREADY_EXPLORED)
+    return;
+  else if (*((int *) (n->xbtdata)) == CURRENTLY_EXPLORING)
+    THROWF(0, 0, "There is a cycle");
+  else {
+    *((int *) (n->xbtdata)) = CURRENTLY_EXPLORING;
+
+    xbt_dynar_foreach(n->out, cursor, edge) {
+      xbt_graph_depth_visit(g, edge->dst, sorted, idx);
+    }
+
+    *((int *) (n->xbtdata)) = ALREADY_EXPLORED;
+    sorted[(*idx)--] = n;
+  }
+}
+
 /********************* Import and Export ******************/
 static xbt_graph_t parsed_graph = NULL;
 static xbt_dict_t parsed_nodes = NULL;
 
-static void *(*__parse_node_label_and_data)(xbt_node_t, const char*, const char*) = NULL;
-static void *(*__parse_edge_label_and_data)(xbt_edge_t, const char*, const char*) = NULL;
+static void *(*__parse_node_label_and_data) (xbt_node_t, const char *,
+                                             const char *) = NULL;
+static void *(*__parse_edge_label_and_data) (xbt_edge_t, const char *,
+                                             const char *) = NULL;
 
 static void __parse_graph_begin(void)
 {
-
-  DEBUG0("<graph>");
-  if(A_graphxml_graph_isDirected == A_graphxml_graph_isDirected_true)
+  XBT_DEBUG("<graph>");
+  if (A_graphxml_graph_isDirected == A_graphxml_graph_isDirected_true)
     parsed_graph = xbt_graph_new_graph(1, NULL);
-  else parsed_graph = xbt_graph_new_graph(0, NULL);
+  else
+    parsed_graph = xbt_graph_new_graph(0, NULL);
 
-  parsed_nodes = xbt_dict_new();
+  parsed_nodes = xbt_dict_new_homogeneous(NULL);
 }
+
 static void __parse_graph_end(void)
 {
   xbt_dict_free(&parsed_nodes);
-  DEBUG0("</graph>");
+  XBT_DEBUG("</graph>");
 }
 
 static void __parse_node(void)
 {
-  xbt_node_t node =
-      xbt_graph_new_node(parsed_graph, NULL);
+  xbt_node_t node = xbt_graph_new_node(parsed_graph, NULL);
 
-  DEBUG1("<node name=\"%s\"/>", A_graphxml_node_name);
-  if(__parse_node_label_and_data)
-    node->data = __parse_node_label_and_data(node,A_graphxml_node_label,
-                                            A_graphxml_node_data);
-  xbt_graph_parse_get_double(&(node->position_x),A_graphxml_node_position_x);
-  xbt_graph_parse_get_double(&(node->position_y),A_graphxml_node_position_y);
+  XBT_DEBUG("<node name=\"%s\"/>", A_graphxml_node_name);
+  if (__parse_node_label_and_data)
+    node->data = __parse_node_label_and_data(node, A_graphxml_node_label,
+                                             A_graphxml_node_data);
+  node->position_x = xbt_graph_parse_get_double(A_graphxml_node_position___x);
+  node->position_y = xbt_graph_parse_get_double(A_graphxml_node_position___y);
 
   xbt_dict_set(parsed_nodes, A_graphxml_node_name, (void *) node, NULL);
 }
 
 static void __parse_edge(void)
 {
-  xbt_edge_t edge =
-    xbt_graph_new_edge(parsed_graph,
-                      xbt_dict_get(parsed_nodes,A_graphxml_edge_source),
-                      xbt_dict_get(parsed_nodes,A_graphxml_edge_target),
-                      NULL);
-
-  if(__parse_edge_label_and_data)
-    edge->data = __parse_edge_label_and_data(edge,A_graphxml_edge_label,
-                                            A_graphxml_edge_data);
-
-  xbt_graph_parse_get_double(&(edge->length),A_graphxml_edge_length);
-
-  DEBUG3("<edge  source=\"%s\" target=\"%s\" length=\"%f\"/>",
-        (char *) (edge->src)->data,
-        (char *) (edge->dst)->data,
-        xbt_graph_edge_get_length(edge));
+  xbt_edge_t edge = xbt_graph_new_edge(parsed_graph,
+                                       xbt_dict_get(parsed_nodes,
+                                                    A_graphxml_edge_source),
+                                       xbt_dict_get(parsed_nodes,
+                                                    A_graphxml_edge_target),
+                                       NULL);
+
+  if (__parse_edge_label_and_data)
+    edge->data = __parse_edge_label_and_data(edge, A_graphxml_edge_label,
+                                             A_graphxml_edge_data);
+
+  edge->length = xbt_graph_parse_get_double(A_graphxml_edge_length);
+
+  XBT_DEBUG("<edge  source=\"%s\" target=\"%s\" length=\"%f\"/>",
+         (char *) (edge->src)->data,
+         (char *) (edge->dst)->data, xbt_graph_edge_get_length(edge));
 }
 
+/** @brief Import a graph from a file following the GraphXML format */
 xbt_graph_t xbt_graph_read(const char *filename,
-                          void *(node_label_and_data)(xbt_node_t, const char*, const char*),
-                          void *(edge_label_and_data)(xbt_edge_t, const char*, const char*))
+                           void *(*node_label_and_data) (xbt_node_t,
+                                                         const char *,
+                                                         const char *),
+                           void *(*edge_label_and_data) (xbt_edge_t,
+                                                         const char *,
+                                                         const char *))
 {
 
   xbt_graph_t graph = NULL;
@@ -514,7 +628,9 @@ xbt_graph_t xbt_graph_read(const char *filename,
   ETag_graphxml_edge_fun = __parse_edge;
 
   xbt_graph_parse_open(filename);
-  xbt_assert1((!xbt_graph_parse()), "Parse error in %s", filename);
+  _XBT_GNUC_UNUSED int res;
+  res = xbt_graph_parse();
+  xbt_assert(!res, "Parse error in %s", filename);
   xbt_graph_parse_close();
 
   graph = parsed_graph;
@@ -523,127 +639,211 @@ xbt_graph_t xbt_graph_read(const char *filename,
   return graph;
 }
 
+/** @brief Export the given graph in the GraphViz formatting for visualization */
 void xbt_graph_export_graphviz(xbt_graph_t g, const char *filename,
-                              const char *(node_name) (xbt_node_t),
-                              const char *(edge_name) (xbt_edge_t))
+                               const char *(node_name) (xbt_node_t),
+                               const char *(edge_name) (xbt_edge_t))
 {
-  int cursor = 0;
+  unsigned int cursor = 0;
   xbt_node_t node = NULL;
   xbt_edge_t edge = NULL;
   FILE *file = NULL;
-  const char *name=NULL;
+  const char *name = NULL;
 
-  file=fopen(filename,"w");
-  xbt_assert1(file, "Failed to open %s \n",filename);
+  file = fopen(filename, "w");
+  xbt_assert(file, "Failed to open %s \n", filename);
 
-  if(g->directed) fprintf(file,"digraph test {\n");
-  else fprintf(file,"graph test {\n");
+  if (g->directed)
+    fprintf(file, "digraph test {\n");
+  else
+    fprintf(file, "graph test {\n");
 
-  fprintf(file,"  graph [overlap=scale]\n");
+  fprintf(file, "  graph [overlap=scale]\n");
+
+  fprintf(file, "  node [shape=box, style=filled]\n");
+  fprintf(file,
+          "  node [width=.3, height=.3, style=filled, color=skyblue]\n\n");
 
-  fprintf(file,"  node [shape=box, style=filled]\n");
-  fprintf(file,"  node [width=.3, height=.3, style=filled, color=skyblue]\n\n");
-  
   xbt_dynar_foreach(g->nodes, cursor, node) {
-    fprintf(file,"  \"%p\" ", node);
-    if((node_name)&&((name=node_name(node)))) fprintf(file,"[label=\"%s\"]",name);
-    fprintf(file,";\n");
+    if (node_name){
+      fprintf(file, "  \"%s\";\n", node_name(node));
+    }else{
+      fprintf(file, "  \"%p\";\n", node);
+    }
   }
   xbt_dynar_foreach(g->edges, cursor, edge) {
-    if(g->directed)
-      fprintf(file,"  \"%p\" -> \"%p\"",edge->src, edge->dst);
-    else
-      fprintf(file,"  \"%p\" -- \"%p\"",edge->src, edge->dst);
-    if((edge_name)&&((name=edge_name(edge)))) fprintf(file,"[label=\"%s\"]",name);
-    fprintf(file,";\n");
+    const char *c;
+    const char *c_dir = "->";
+    const char *c_ndir = "--";
+    if (g->directed){
+      c = c_dir;
+    }else{
+      c = c_ndir;
+    }
+    const char *src_name, *dst_name;
+    if (node_name){
+      src_name = node_name(edge->src);
+      dst_name = node_name(edge->dst);
+      fprintf(file, "  \"%s\" %s \"%s\"", src_name, c, dst_name);
+    }else{
+      fprintf(file, "  \"%p\" %s \"%p\"", edge->src, c, edge->dst);
+    }
+
+    if ((edge_name) && ((name = edge_name(edge))))
+      fprintf(file, "[label=\"%s\"]", name);
+    fprintf(file, ";\n");
   }
-  fprintf(file,"}\n");
+  fprintf(file, "}\n");
   fclose(file);
 }
 
+/** @brief Export the given graph in the GraphXML format */
 void xbt_graph_export_graphxml(xbt_graph_t g, const char *filename,
-                              const char *(node_name)(xbt_node_t),
-                              const char *(edge_name)(xbt_edge_t),
-                              const char *(node_data_print)(void *),
-                              const char *(edge_data_print)(void *))
+                               const char *(node_name) (xbt_node_t),
+                               const char *(edge_name) (xbt_edge_t),
+                               const char *(node_data_print) (void *),
+                               const char *(edge_data_print) (void *))
 {
-  int cursor = 0;
+  unsigned int cursor = 0;
   xbt_node_t node = NULL;
   xbt_edge_t edge = NULL;
   FILE *file = NULL;
   const char *name = NULL;
 
-  file=fopen(filename,"w");
-  xbt_assert1(file, "Failed to open %s \n",filename);
+  file = fopen(filename, "w");
+  xbt_assert(file, "Failed to open %s \n", filename);
 
-  fprintf(file,"<?xml version='1.0'?>\n");
-  fprintf(file,"<!DOCTYPE graph SYSTEM \"graphxml.dtd\">\n");
-  if(g->directed) fprintf(file,"<graph isDirected=\"true\">\n");
-  else fprintf(file,"<graph isDirected=\"false\">\n");
+  fprintf(file, "<?xml version='1.0'?>\n");
+  fprintf(file, "<!DOCTYPE graph SYSTEM \"graphxml.dtd\">\n");
+  if (g->directed)
+    fprintf(file, "<graph isDirected=\"true\">\n");
+  else
+    fprintf(file, "<graph isDirected=\"false\">\n");
   xbt_dynar_foreach(g->nodes, cursor, node) {
-    fprintf(file,"  <node name=\"%p\" ", node);
-    if((node_name)&&((name=node_name(node)))) fprintf(file,"label=\"%s\" ",name);
-    if((node_data_print)&&((name=node_data_print(node->data)))) 
-      fprintf(file,"data=\"%s\" ",name);
-    fprintf(file,">\n");
+    fprintf(file, "  <node name=\"%p\" ", node);
+    if ((node_name) && ((name = node_name(node))))
+      fprintf(file, "label=\"%s\" ", name);
+    if ((node_data_print) && ((name = node_data_print(node->data))))
+      fprintf(file, "data=\"%s\" ", name);
+    fprintf(file, ">\n");
   }
   xbt_dynar_foreach(g->edges, cursor, edge) {
-    fprintf(file,"  <edge source=\"%p\" target =\"%p\" ",
-           edge->src, edge->dst );
-    if((edge_name)&&((name=edge_name(edge)))) fprintf(file,"label=\"%s\" ",name);
-    if(edge->length>=0.0) fprintf(file,"length=\"%g\" ",edge->length);
-    if((edge_data_print)&&((name=edge_data_print(edge->data)))) 
-      fprintf(file,"data=\"%s\" ",name);
-    fprintf(file,">\n");
+    fprintf(file, "  <edge source=\"%p\" target =\"%p\" ",
+            edge->src, edge->dst);
+    if ((edge_name) && ((name = edge_name(edge))))
+      fprintf(file, "label=\"%s\" ", name);
+    if (edge->length >= 0.0)
+      fprintf(file, "length=\"%g\" ", edge->length);
+    if ((edge_data_print) && ((name = edge_data_print(edge->data))))
+      fprintf(file, "data=\"%s\" ", name);
+    fprintf(file, ">\n");
   }
-  fprintf(file,"</graph>\n");
+  fprintf(file, "</graph>\n");
   fclose(file);
 }
 
-xbt_node_t* xbt_graph_topo_sort(xbt_graph_t g)
+/** @brief Load a graph from a file (in the SimGrid Graph format) */
+xbt_graph_t xbt_graph_load (const char *filename)
 {
-  xbt_node_t* sorted;
-  int cursor,idx;
-  xbt_node_t node;
- unsigned long n;
-
- n= xbt_dynar_length(g->nodes); 
- idx=n-1;
-
-  sorted=xbt_malloc(n*sizeof(xbt_node_t));
-  xbt_dynar_foreach(g->nodes,cursor , node)
-    {
-      node->xbtdata=xbt_new0(int,1);
-    }
+  FILE *file = NULL;
+  ssize_t read;
+  file = fopen (filename, "r");
+  xbt_assert(file, "Failed to open %s \n", filename);
+
+  xbt_dict_t nodes_dict = xbt_dict_new_homogeneous(NULL);
+  xbt_graph_t ret = xbt_graph_new_graph (0, NULL);
+
+  //read the number of nodes
+  size_t size;
+  char *nnodes_str = NULL;
+  read = getline (&nnodes_str, &size, file);
+  if (read == -1)
+    THROWF(system_error, 0, "getline failed to read the number of nodes (errno = %d)", errno);
+  int i, nnodes = atoi (nnodes_str);
+  free (nnodes_str);
+
+  //read all nodes
+  for (i = 0; i < nnodes; i++){
+    char *node_str = NULL;
+    read = getline (&node_str, &size, file);
+    if (read == -1)
+      THROWF(system_error, 0, "getline failed to read all nodes (errno = %d)", errno);
+    xbt_node_t n;
+    char *name = xbt_strdup (node_str);
+    xbt_str_subst (name, '\n', '\0', 0);
+    n = xbt_graph_new_node (ret, name);
+    xbt_dict_set (nodes_dict, name, n, NULL);
+    free (node_str);
+  }
 
-  xbt_dynar_foreach(g->nodes,cursor , node) 
-    { 
-      xbt_graph_depth_visit(g,node,sorted,&idx);   
-    }
-         
-  return sorted; 
+  //read the number of edges
+  char *nedges_str = NULL;
+  read = getline (&nedges_str, &size, file);
+  if (read == -1)
+    THROWF(system_error, 0, "getline failed to read the number of edges (errno = %d)", errno);
+  int nedges = atoi (nedges_str);
+  free (nedges_str);
+
+  //read all edges
+  for (i = 0; i < nedges; i++){
+    char *edge_str = NULL, edge_id[200], node_source[200], node_target[200];
+    read = getline (&edge_str, &size, file);
+    if (read == -1)
+      THROWF(system_error, 0, "getline failed to read all edges (errno = %d)", errno);
+    sscanf (edge_str, "%s %s %s", edge_id, node_source, node_target);
+    free (edge_str);
+    xbt_str_subst (edge_id, '\n', '\0', 0);
+    xbt_str_subst (node_source, '\n', '\0', 0);
+    xbt_str_subst (node_target, '\n', '\0', 0);
+
+    xbt_node_t source = xbt_dict_get (nodes_dict, node_source);
+    xbt_node_t target = xbt_dict_get (nodes_dict, node_target);
+    xbt_graph_new_edge (ret, source, target, xbt_strdup(edge_id));
+  }
+  xbt_dict_free (&nodes_dict);
+  return ret;
 }
 
-void xbt_graph_depth_visit(xbt_graph_t g,xbt_node_t n,xbt_node_t* sorted,int* idx )
+/** @brief Save a graph from a file (in the SimGrid Graph format) */
+void xbt_graph_save (xbt_graph_t span,
+                     const char *filename,
+                     const char *(nname) (xbt_node_t),
+                     const char *(ename) (xbt_edge_t))
 {
-  int cursor;
+  FILE *file = NULL;
+  file = fopen(filename, "w");
+  xbt_assert(file, "Failed to open %s \n", filename);
+
+  xbt_dynar_t nodes = xbt_graph_get_nodes (span);
+  xbt_dynar_t edges = xbt_graph_get_edges (span);
+  unsigned int cpt;
+  xbt_node_t node;
+  fprintf (file, "%lu\n", xbt_dynar_length (nodes));
+  xbt_dynar_foreach (nodes, cpt, node) {
+    if (nname){
+      fprintf (file, "%s\n", nname(node));
+    }else{
+      fprintf (file, "%p\n", node);
+    }
+  }
+  fprintf (file, "%lu\n", xbt_dynar_length (edges));
   xbt_edge_t edge;
-  if (*((int*)(n->xbtdata))==ALREADY_EXPLORED)
-    return;
-  else if (*((int*)(n->xbtdata))==CURRENTLY_EXPLORING)
-    THROW0(0,0,"There is a cycle");
-  else
-    {
-      *((int*)(n->xbtdata))=CURRENTLY_EXPLORING;
-
-      xbt_dynar_foreach(n->out,cursor, edge)
-       {
-         xbt_graph_depth_visit(g,edge->dst,sorted,idx);
-       }
-      *((int*)(n->xbtdata))=ALREADY_EXPLORED;
-      sorted[(*idx)--]=n;
-    }       
+  xbt_dynar_foreach (edges, cpt, edge) {
+    xbt_node_t source = xbt_graph_edge_get_source (edge);
+    xbt_node_t target = xbt_graph_edge_get_target (edge);
+    if (ename){
+      if (nname){
+        fprintf (file, "%s %s %s\n", ename(edge), nname(source), nname(target));
+      }else{
+        fprintf (file, "%s %p %p\n", ename(edge), source, target);
+      }
+    }else{
+      if (nname){
+        fprintf (file, "%p %s %s\n", edge, nname(source), nname(target));
+      }else{
+        fprintf (file, "%p %p %p\n", edge, source, target);
+      }
+    }
+  }
+  fclose (file);
 }