number of unsatisfied dependencies can be negative, using an unsigned

int leads to weird outputs by SD_task_dump

diff --git a/src/simdag/private.h b/src/simdag/private.h
index db39ef4..1010a54 100644 (file)
--- a/src/simdag/private.h
+++ b/src/simdag/private.h
@@ -89,7 +89,7 @@ typedef struct SD_task {
   /* dependencies */
   xbt_dynar_t tasks_before;
   xbt_dynar_t tasks_after;
   /* dependencies */
   xbt_dynar_t tasks_before;
   xbt_dynar_t tasks_after;

-  unsigned int unsatisfied_dependencies;
+  int unsatisfied_dependencies;

   unsigned int is_not_ready;
 
   /* scheduling parameters (only exist in state SD_SCHEDULED) */
   unsigned int is_not_ready;
 
   /* scheduling parameters (only exist in state SD_SCHEDULED) */

diff --git a/src/simdag/sd_task.c b/src/simdag/sd_task.c
index 8cd18a8..3673944 100644 (file)
--- a/src/simdag/sd_task.c
+++ b/src/simdag/sd_task.c
@@ -509,7 +509,7 @@ void SD_task_dump(SD_task_t task)
     }
   }
   XBT_INFO("  - amount: %.0f", SD_task_get_amount(task));
     }
   }
   XBT_INFO("  - amount: %.0f", SD_task_get_amount(task));

-  XBT_INFO("  - Dependencies to satisfy: %u", task->unsatisfied_dependencies);
+  XBT_INFO("  - Dependencies to satisfy: %d", task->unsatisfied_dependencies);

   if (!xbt_dynar_is_empty(task->tasks_before)) {
     XBT_INFO("  - pre-dependencies:");
     xbt_dynar_foreach(task->tasks_before, counter, dependency) {
   if (!xbt_dynar_is_empty(task->tasks_before)) {
     XBT_INFO("  - pre-dependencies:");
     xbt_dynar_foreach(task->tasks_before, counter, dependency) {