Replace NULL with nullptr.

[loba.git] / options.cpp

diff --git a/options.cpp b/options.cpp
index 21f9b9b8650a795a652ca216cb15ac8f0d2bed04..006c83a9ffe966959e221ba6a81c54e580019353 100644 (file)
--- a/options.cpp
+++ b/options.cpp
@@ -18,6 +18,7 @@ XBT_LOG_EXTERNAL_DEFAULT_CATEGORY(main);
 #include "loba_makhoul.h"
 #include "loba_makhoul2.h"
 #include "loba_makhoul3.h"
+#include "loba_pstar.h"
 #include "loba_simple.h"
 #include "misc.h"
 
@@ -80,7 +81,7 @@ namespace opt {
     unsigned lb_maxiter = 0;
     unsigned comp_maxiter = 0;
     double time_limit = 0;
-    bool exit_on_convergence = false;
+    unsigned exit_on_convergence = 0;
     bool exit_on_close = true;
 
     // Named parameters lists
@@ -105,6 +106,8 @@ namespace opt {
                    loba_makhoul3);
         NOL_INSERT("none", "no load-balancing (for testing only)",
                    process);
+        NOL_INSERT("pstar", "balance with pstar strategy",
+                   loba_pstar);
         NOL_INSERT("simple", "balance with least loaded neighbor",
                    loba_simple);
     }
@@ -166,25 +169,24 @@ const char* opt_helper::on_off(bool b)
 
 const char* opt_helper::descr(const char* str)
 {
-    std::string& res = descr_str;
-    res = str;
-    res.resize(DATA_DESCR_WIDTH, '.');
-    return res.c_str();
+    descr_str = str;
+    descr_str.push_back(' ');
+    descr_str.resize(DATA_DESCR_WIDTH, '.');
+    return descr_str.c_str();
 }
 
 template <typename T>
 const char* opt_helper::val_or_string(const T& val, const char* str,
                                       const T& deflt)
 {
-    std::string& res = val_or_string_str;
     if (val != deflt) {
         std::ostringstream oss;
         oss << val;
-        res = oss.str();
+        val_or_string_str = oss.str();
     } else {
-        res = str;
+        val_or_string_str = str;
     }
-    return res.c_str();
+    return val_or_string_str.c_str();
 }
 
 template <typename T>
@@ -234,7 +236,7 @@ bool opt::parse_args(int* argc, char* argv[])
     opterr = 0;
     while ((c = getopt(*argc, argv,
                        "%:"
-                       "a:bc:C:d:D:eEGhi:I:k:l:L:m:M:N:r:Rs:S:t:T:vVx:X:Z"))
+                       "a:bc:C:d:D:eEg:hi:I:k:l:L:m:M:N:r:Rs:S:t:T:vVx:X:Z"))
            != -1) {
         switch (c) {
         case '%':
@@ -280,8 +282,8 @@ bool opt::parse_args(int* argc, char* argv[])
         case 'E':
             opt::egocentric = !opt::egocentric;
             break;
-        case 'G':
-            opt::exit_on_convergence = !opt::exit_on_convergence;
+        case 'g':
+            PARSE_ARG(opt::exit_on_convergence);
             break;
         case 'h':
             opt::help_requested++;
@@ -393,7 +395,7 @@ bool opt::parse_args(int* argc, char* argv[])
     }
 
     if (!opt::auto_depl::random_seed)
-        opt::auto_depl::random_seed = time(NULL);
+        opt::auto_depl::random_seed = time(nullptr);
 
     return result;
 }
@@ -447,7 +449,8 @@ void opt::print()
           h.val_or_string(comp_maxiter, "no limit"));
     DESCR("convergence is assumed within (\%)", "%g", opt::avg_load_ratio);
     DESCR("time limit", "%s", h.val_or_string(time_limit, "no limit"));
-    DESCR("exit on convergence", "%s", h.on_off(exit_on_convergence));
+    DESCR("exit on convergence", "%s",
+          h.val_or_string(exit_on_convergence, "disabled"));
     DESCR("exit on close", "%s", h.on_off(exit_on_close));
     XBT_INFO("`----");
 
@@ -571,9 +574,13 @@ void opt::usage()
               << " [" << opt::avg_load_ratio << "]\n";
     if (opt::help_requested > 1)
         std::clog << o("")
-                  << "or, if negative, 100 * (nhosts / -value) / avg_load\n";
-    std::clog << o("-G") << "toggle exit on detection of convergence"
-              << " [" << opt_helper::on_off(opt::exit_on_convergence) << "]\n";
+                  << "- if negative: use 100 * (nhosts / -value) / avg_load\n";
+    std::clog << o("-g value")
+              << "number of consecutive detections of convergence to exit"
+              << " [" << opt::exit_on_convergence << "]\n";
+    if (opt::help_requested > 1)
+        std::clog << o("")
+                  << "- use 0 to disable\n";
     std::clog << o("-t value")
               << "time limit (simulated time), 0 for no limit"
               << " [" << opt::time_limit << "]\n";
@@ -591,7 +598,7 @@ void opt::usage()
 
     std::clog << "\nLogging support:\n"
               << "    See SimGrid documentation on:\n"
-              << "        http://simgrid.gforge.inria.fr/doc/group__XBT__log.html#log_user\n"
+              << "        http://simgrid.gforge.inria.fr/simgrid/latest/doc/outcomes_logs.html#log_user\n"
               << "    Existing categories are:\n"
               << "        simu : root of following categories\n"
               << "        main : messages from global infrastructure\n"