Stupid error.

[simgrid.git] / src / simix / smx_deployment.c

diff --git a/src/simix/smx_deployment.c b/src/simix/smx_deployment.c
index 2476a4b..6c7d332 100644 (file)
--- a/src/simix/smx_deployment.c
+++ b/src/simix/smx_deployment.c
@@ -4,7 +4,7 @@
 /* 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 "private.h"
+#include "smx_private.h"
 #include "xbt/sysdep.h"
 #include "xbt/log.h"
 #include "xbt/dict.h"
@@ -19,6 +19,8 @@ static char *parse_host = NULL;
 static double start_time = 0.0;
 static double kill_time = -1.0;
 
+extern int surf_parse_lineno;
+
 static void parse_process_init(void)
 {
   smx_host_t host = SIMIX_host_get_by_name(A_surfxml_process_host);
@@ -28,26 +30,24 @@ static void parse_process_init(void)
   parse_code = SIMIX_get_registered_function(A_surfxml_process_function);
   xbt_assert(parse_code, "Function '%s' unknown",
               A_surfxml_process_function);
-  parse_argc = 0;
-  parse_argv = NULL;
-  parse_argc++;
-  parse_argv = xbt_realloc(parse_argv, (parse_argc) * sizeof(char *));
-  parse_argv[(parse_argc) - 1] = xbt_strdup(A_surfxml_process_function);
+  parse_argv = xbt_new(char *, 2);
+  parse_argv[0] = xbt_strdup(A_surfxml_process_function);
+  parse_argc = 1;
   start_time= surf_parse_get_double(A_surfxml_process_start_time);
   kill_time = surf_parse_get_double(A_surfxml_process_kill_time);
 }
 
 static void parse_argument(void)
 {
-  parse_argc++;
-  parse_argv = xbt_realloc(parse_argv, (parse_argc) * sizeof(char *));
-  parse_argv[(parse_argc) - 1] = xbt_strdup(A_surfxml_argument_value);
+  parse_argv = xbt_realloc(parse_argv, (parse_argc + 2) * sizeof(char *));
+  parse_argv[parse_argc++] = xbt_strdup(A_surfxml_argument_value);
 }
 
 static void parse_process_finalize(void)
 {
   smx_process_arg_t arg = NULL;
   smx_process_t process = NULL;
+  parse_argv[parse_argc] = NULL;
   if (start_time > SIMIX_get_clock()) {
     arg = xbt_new0(s_smx_process_arg_t, 1);
     arg->name = parse_argv[0];
@@ -73,7 +73,7 @@ static void parse_process_finalize(void)
                                             parse_argv,
                                             current_property_set);
     else
-      SIMIX_req_process_create(&process, parse_argv[0], parse_code, NULL, parse_host, parse_argc, parse_argv,
+      simcall_process_create(&process, parse_argv[0], parse_code, NULL, parse_host, parse_argc, parse_argv,
                                current_property_set);
     /* verify if process has been created (won't be the case if the host is currently dead, but that's fine) */
     if (!process) {
@@ -104,6 +104,8 @@ static void parse_process_finalize(void)
  */
 void SIMIX_launch_application(const char *file)
 {
+  xbt_ex_t e;
+
   _XBT_GNUC_UNUSED int parse_status;
   xbt_assert(simix_global,
               "SIMIX_global_init has to be called before SIMIX_launch_application.");
@@ -117,9 +119,14 @@ void SIMIX_launch_application(const char *file)
                        parse_process_finalize);
 
   surf_parse_open(file);
-  parse_status = surf_parse();
-  surf_parse_close();
-  xbt_assert(!parse_status, "Parse error in %s", file);
+  TRY {
+         parse_status = surf_parse();
+         surf_parse_close();
+         xbt_assert(!parse_status, "Parse error at %s:%d", file,surf_parse_lineno);
+  } CATCH(e) {
+         xbt_die("Unrecoverable error at %s:%d: %s",
+                  file, surf_parse_lineno, e.msg);
+  }
 }
 
 /**
@@ -168,7 +175,7 @@ xbt_main_func_t SIMIX_get_registered_function(const char *name)
   xbt_assert(simix_global,
               "SIMIX_global_init has to be called before SIMIX_get_registered_function.");
 
-  res = xbt_dict_get_or_null(simix_global->registered_functions, name);
+  res = (xbt_main_func_t)xbt_dict_get_or_null(simix_global->registered_functions, name);
   return res ? res : default_function;
 }
 
@@ -194,20 +201,18 @@ void SIMIX_process_set_function(const char *process_host,
   parse_code = SIMIX_get_registered_function(process_function);
   xbt_assert(parse_code, "Function '%s' unknown", process_function);
 
-  parse_argc = 0;
-  parse_argv = NULL;
-  parse_argc++;
-  parse_argv = xbt_realloc(parse_argv, (parse_argc) * sizeof(char *));
-  parse_argv[(parse_argc) - 1] = xbt_strdup(process_function);
-  start_time = process_start_time;
-  kill_time = process_kill_time;
+  parse_argc = 1 + xbt_dynar_length(arguments);
+  parse_argv = xbt_new(char *, parse_argc + 1);
+  parse_argv[0] = xbt_strdup(process_function);
 
   /* add arguments */
   xbt_dynar_foreach(arguments, i, arg) {
-    parse_argc++;
-    parse_argv = xbt_realloc(parse_argv, (parse_argc) * sizeof(char *));
-    parse_argv[(parse_argc) - 1] = xbt_strdup(arg);
+    parse_argv[i + 1] = xbt_strdup(arg);
   }
+  parse_argv[parse_argc] = NULL;
+
+  start_time = process_start_time;
+  kill_time = process_kill_time;
 
   /* finalize */
   parse_process_finalize();