X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/502c1fb7b48f799d36d2ee04d978b3bb54fb0d00..6229ff3a39c8fb195846ea9eb67cf071654de597:/src/msg/msg_actions.c

diff --git a/src/msg/msg_actions.c b/src/msg/msg_actions.c
index b7a54c516d..5ac10d13da 100644
--- a/src/msg/msg_actions.c
+++ b/src/msg/msg_actions.c
@@ -4,7 +4,8 @@
 /* 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 "simgrid_config.h" //For getline, keep that include first
+
 #include "msg/private.h"
 #include "xbt/str.h"
 #include "xbt/dynar.h"
@@ -21,7 +22,7 @@ static FILE *action_fp = NULL;
 static char *action_line = NULL;
 static size_t action_len = 0;
 
-static char*const* action_get_action(char *name);
+static const char **action_get_action(char *name);
 
 /** \ingroup msg_actions
  * \brief Registers a function to handle a kind of action
@@ -51,16 +52,16 @@ void MSG_action_unregister(const char *action_name)
 
 static int MSG_action_runner(int argc, char *argv[])
 {
-  char *const*evt;
+  const char **evt;
   if (action_fp) {              // A unique trace file
 
     while ((evt = action_get_action(argv[0]))) {
       msg_action_fun function = xbt_dict_get(action_funs, evt[1]);
       (*function) (evt);
-      free((char**)evt);
+      free(evt);
     }
   } else {                      // Should have got my trace file in argument
-    xbt_assert1(argc >= 2,
+    xbt_assert(argc >= 2,
                 "No '%s' agent function provided, no simulation-wide trace file provided to MSG_action_trace_run(), "
                 "and no process-wide trace file provided in deployment file. Aborting.",
                 argv[0]
@@ -70,9 +71,9 @@ static int MSG_action_runner(int argc, char *argv[])
       if (!strcmp(argv[0],evt[0])) {
         msg_action_fun function = xbt_dict_get(action_funs, evt[1]);
         (*function) (evt);
-        free((char**)evt);
+        free(evt);
       } else {
-        WARN1("%s: Ignore trace element not for me",
+        XBT_WARN("%s: Ignore trace element not for me",
               xbt_replay_trace_reader_position(reader));
       }
     }
@@ -95,7 +96,7 @@ void _MSG_action_exit()
 }
 
 
-static char*const* action_get_action(char *name)
+static const char **action_get_action(char *name)
 {
   ssize_t read;
   xbt_dynar_t evt = NULL;
@@ -170,18 +171,18 @@ MSG_error_t MSG_action_trace_run(char *path)
 
   if (path) {
     action_fp = fopen(path, "r");
-    xbt_assert2(action_fp != NULL, "Cannot open %s: %s", path,
+    xbt_assert(action_fp != NULL, "Cannot open %s: %s", path,
                 strerror(errno));
   }
   res = MSG_main();
 
   if (xbt_dict_size(action_queues)) {
-    WARN0
+    XBT_WARN
         ("Not all actions got consumed. If the simulation ended successfully (without deadlock), you may want to add new processes to your deployment file.");
 
 
     xbt_dict_foreach(action_queues, cursor, name, todo) {
-      WARN2("Still %lu actions for %s", xbt_dynar_length(todo), name);
+      XBT_WARN("Still %lu actions for %s", xbt_dynar_length(todo), name);
     }
   }