Be more picky when parsing options.

[loba.git] / main.cpp

diff --git a/main.cpp b/main.cpp
index 9bffd8b2fbdc8d35fc3de9c0bce6229831f1f740..ad59642611707298e6e9e335e23843644a122911 100644 (file)
--- a/main.cpp
+++ b/main.cpp
@@ -36,8 +36,9 @@ namespace {
         EXIT_FAILURE_CLEAN = 0x08,  // error at cleanup
     };
 
         EXIT_FAILURE_CLEAN = 0x08,  // error at cleanup
     };
 

-    mutex_t proc_mutex;
-    condition_t proc_cond;
+    // Cannot be globally initialized...
+    mutex_t* proc_mutex;
+    condition_t* proc_cond;

     unsigned proc_counter = 0;
 
     statistics comps;
     unsigned proc_counter = 0;
 
     statistics comps;


@@ -52,13 +53,13 @@ static int simulation_main(int argc, char* argv[])
     try {
         proc = opt::loba_algorithms.new_instance(opt::loba_algo, argc, argv);
 
     try {
         proc = opt::loba_algorithms.new_instance(opt::loba_algo, argc, argv);
 

-        proc_mutex.acquire();
+        proc_mutex->acquire();

         ++proc_counter;
         ++proc_counter;

-        proc_mutex.release();
+        proc_mutex->release();

 
         result = proc->run();
 
 
         result = proc->run();
 

-        proc_mutex.acquire();
+        proc_mutex->acquire();

         comps.push(proc->get_comp());
         loads.push(proc->get_real_load());
 
         comps.push(proc->get_comp());
         loads.push(proc->get_real_load());
 


@@ -68,23 +69,25 @@ static int simulation_main(int argc, char* argv[])
         // destroys a communication they had together.
 
         --proc_counter;
         // destroys a communication they had together.
 
         --proc_counter;

-        proc_cond.broadcast();
+        proc_cond->broadcast();

         while (proc_counter > 0)
         while (proc_counter > 0)

-            proc_cond.wait(proc_mutex);
-        proc_mutex.release();
+            proc_cond->wait(*proc_mutex);
+        proc_mutex->release();

 
         delete proc;
     }
 
         delete proc;
     }

-    catch (std::invalid_argument& e) {
+    catch (const std::invalid_argument& e) {

         THROW1(arg_error, 0, "%s", e.what());
     }
         THROW1(arg_error, 0, "%s", e.what());
     }

+    catch (const std::exception& e) {
+        THROW1(0, 0, "%s", e.what());
+    }

     return result;
 }
 
 static void check_for_lost_load()
 {
     double total_init = process::get_total_load_init();
     return result;
 }
 
 static void check_for_lost_load()
 {
     double total_init = process::get_total_load_init();

-

     double total_exit = process::get_total_load_exit();
     double lost = total_init - total_exit;
     double lost_ratio = 100.0 * lost / total_init;
     double total_exit = process::get_total_load_exit();
     double lost = total_init - total_exit;
     double lost_ratio = 100.0 * lost / total_init;


@@ -192,12 +195,18 @@ int main(int argc, char* argv[])
 
         exit_status = EXIT_FAILURE_SIMU; // =====
 
 
         exit_status = EXIT_FAILURE_SIMU; // =====
 

+        proc_mutex = new mutex_t();
+        proc_cond = new condition_t();
+

         // Launch the MSG simulation.
         XBT_INFO("Starting simulation at %f...", MSG_get_clock());
         res = MSG_main();
         simulated_time = MSG_get_clock();
         XBT_INFO("Simulation ended at %f.", simulated_time);
 
         // Launch the MSG simulation.
         XBT_INFO("Starting simulation at %f...", MSG_get_clock());
         res = MSG_main();
         simulated_time = MSG_get_clock();
         XBT_INFO("Simulation ended at %f.", simulated_time);
 

+        delete proc_cond;
+        delete proc_mutex;
+

         if (res != MSG_OK)
             THROW1(0, 0, "MSG_main() failed with status %#x", res);
 
         if (res != MSG_OK)
             THROW1(0, 0, "MSG_main() failed with status %#x", res);