Remove explicit conversion to std::string when it's not required.

[simgrid.git] / docs / source / tuto_s4u / master-workers-lab4.cpp

diff --git a/docs/source/tuto_s4u/master-workers-lab4.cpp b/docs/source/tuto_s4u/master-workers-lab4.cpp
index 3cc8ff2..6973e69 100644 (file)
--- a/docs/source/tuto_s4u/master-workers-lab4.cpp
+++ b/docs/source/tuto_s4u/master-workers-lab4.cpp
@@ -18,8 +18,8 @@ XBT_LOG_NEW_DEFAULT_CATEGORY(s4u_app_masterworker, "Messages specific for this e
 
 static void worker(std::string category)
 {
-  const std::string mailbox_name   = std::string("worker-") + std::to_string(simgrid::s4u::this_actor::get_pid());
-  simgrid::s4u::Mailbox* mailbox   = simgrid::s4u::Mailbox::by_name(mailbox_name);
+  const std::string mailbox_name = "worker-" + std::to_string(simgrid::s4u::this_actor::get_pid());
+  simgrid::s4u::Mailbox* mailbox = simgrid::s4u::Mailbox::by_name(mailbox_name);
 
   while (true) { // Master forcefully kills the workers by the end of the simulation
     double* msg         = mailbox->get<double>();
@@ -45,22 +45,21 @@ static void master(std::vector<std::string> args)
   std::vector<simgrid::s4u::ActorPtr> actors;
 
   simgrid::s4u::Engine* e = simgrid::s4u::Engine::get_instance();
-  std::string my_name     = std::string("master-") + std::to_string(simgrid::s4u::this_actor::get_pid());
+  std::string my_name     = "master-" + std::to_string(simgrid::s4u::this_actor::get_pid());
 
   XBT_INFO("Asked to run for %.1f seconds", simulation_duration);
 
   for (auto* host : e->get_all_hosts()) {
-    simgrid::s4u::ActorPtr act =
-        simgrid::s4u::Actor::create(std::string("Worker-") + host->get_name(), host, worker, my_name);
+    simgrid::s4u::ActorPtr act = simgrid::s4u::Actor::create("Worker-" + host->get_name(), host, worker, my_name);
     actors.push_back(act);
   }
 
   int task_id = 0;
   while (simgrid::s4u::Engine::get_clock() < simulation_duration) { /* For each task: */
     /* - Select a worker in a round-robin way */
-    aid_t worker_pid                 = actors.at(task_id % actors.size())->get_pid();
-    std::string mailbox_name         = std::string("worker-") + std::to_string(worker_pid);
-    simgrid::s4u::Mailbox* mailbox   = simgrid::s4u::Mailbox::by_name(mailbox_name);
+    aid_t worker_pid               = actors.at(task_id % actors.size())->get_pid();
+    std::string mailbox_name       = "worker-" + std::to_string(worker_pid);
+    simgrid::s4u::Mailbox* mailbox = simgrid::s4u::Mailbox::by_name(mailbox_name);
 
     /* - Send the computation cost to that worker */
     XBT_DEBUG("Sending task %d to mailbox '%s'", task_id, mailbox->get_cname());