Use Mailbox::get_unique<>(), and save a few delete.

[simgrid.git] / examples / s4u / app-pingpong / s4u-app-pingpong.cpp

diff --git a/examples/s4u/app-pingpong/s4u-app-pingpong.cpp b/examples/s4u/app-pingpong/s4u-app-pingpong.cpp
index 03fa89a..8b4ed86 100644 (file)
--- a/examples/s4u/app-pingpong/s4u-app-pingpong.cpp
+++ b/examples/s4u/app-pingpong/s4u-app-pingpong.cpp
@@ -1,4 +1,4 @@
-/* Copyright (c) 2007-2019. The SimGrid Team. All rights reserved.          */
+/* Copyright (c) 2007-2020. The SimGrid Team. All rights reserved.          */
 
 /* 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. */
@@ -11,18 +11,16 @@ static void pinger(simgrid::s4u::Mailbox* mailbox_in, simgrid::s4u::Mailbox* mai
 {
   XBT_INFO("Ping from mailbox %s to mailbox %s", mailbox_in->get_name().c_str(), mailbox_out->get_name().c_str());
 
-  /* - Do the ping with a 1-Byte task (latency bound) ... */
-  double* payload = new double();
-  *payload        = simgrid::s4u::Engine::get_clock();
+  /* - Do the ping with a 1-Byte payload (latency bound) ... */
+  auto* payload = new double(simgrid::s4u::Engine::get_clock());
 
   mailbox_out->put(payload, 1);
   /* - ... then wait for the (large) pong */
-  const double* sender_time = static_cast<double*>(mailbox_in->get());
+  auto sender_time = mailbox_in->get_unique<double>();
 
   double communication_time = simgrid::s4u::Engine::get_clock() - *sender_time;
-  XBT_INFO("Task received : large communication (bandwidth bound)");
+  XBT_INFO("Payload received : large communication (bandwidth bound)");
   XBT_INFO("Pong time (bandwidth bound): %.3f", communication_time);
-  delete sender_time;
 }
 
 static void ponger(simgrid::s4u::Mailbox* mailbox_in, simgrid::s4u::Mailbox* mailbox_out)
@@ -30,16 +28,14 @@ static void ponger(simgrid::s4u::Mailbox* mailbox_in, simgrid::s4u::Mailbox* mai
   XBT_INFO("Pong from mailbox %s to mailbox %s", mailbox_in->get_name().c_str(), mailbox_out->get_name().c_str());
 
   /* - Receive the (small) ping first ....*/
-  const double* sender_time = static_cast<double*>(mailbox_in->get());
+  auto sender_time          = mailbox_in->get_unique<double>();
   double communication_time = simgrid::s4u::Engine::get_clock() - *sender_time;
-  XBT_INFO("Task received : small communication (latency bound)");
-  XBT_INFO(" Ping time (latency bound) %f", communication_time);
-  delete sender_time;
+  XBT_INFO("Payload received : small communication (latency bound)");
+  XBT_INFO("Ping time (latency bound) %f", communication_time);
 
   /*  - ... Then send a 1GB pong back (bandwidth bound) */
-  double* payload = new double();
-  *payload        = simgrid::s4u::Engine::get_clock();
-  XBT_INFO("task_bw->data = %.3f", *payload);
+  auto* payload = new double(simgrid::s4u::Engine::get_clock());
+  XBT_INFO("payload = %.3f", *payload);
 
   mailbox_out->put(payload, 1e9);
 }