X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/d4abf745ddd1a9ad9480003c42964d55ec966f29..321102577020b194dfb7ba89e48687952816849e:/src/surf/HostImpl.cpp

diff --git a/src/surf/HostImpl.cpp b/src/surf/HostImpl.cpp
index 488aca5f7f..7f1e1022a1 100644
--- a/src/surf/HostImpl.cpp
+++ b/src/surf/HostImpl.cpp
@@ -37,22 +37,27 @@ HostImpl::~HostImpl()
 {
   /* All actors should be gone when the host is turned off (by the end of the simulation). */
   if (not actor_list_.empty()) {
-    std::string msg = "Shutting down host, but it's not empty:";
-    for (auto const& actor : actor_list_)
-      msg += "\n\t" + std::string(actor.get_name());
-
-    EngineImpl::get_instance()->display_all_actor_status();
-    xbt_die("%s", msg.c_str());
+    const char* msg = "Shutting down host, but it's not empty";
+    try {
+      std::string actors;
+      for (auto const& actor : actor_list_)
+        actors += "\n\t" + std::string(actor.get_name());
+
+      EngineImpl::get_instance()->display_all_actor_status();
+      xbt_die("%s:%s", msg, actors.c_str());
+    } catch (const std::bad_alloc& ba) {
+      xbt_die("%s (cannot print actor list: %s)", msg, ba.what());
+    }
   }
   for (auto const& arg : actors_at_boot_)
     delete arg;
   actors_at_boot_.clear();
 
-  for (auto const& d : disks_)
-    d.second->destroy();
+  for (auto const& [_, d] : disks_)
+    d->destroy();
 
-  for (auto const& vm : vms_)
-    vm.second->vm_destroy();
+  for (auto const& [_, vm] : vms_)
+    vm->vm_destroy();
 }
 
 /** @brief Fire the required callbacks and destroy the object
@@ -81,10 +86,10 @@ void HostImpl::turn_on() const
 void HostImpl::turn_off(const actor::ActorImpl* issuer)
 {
   /* turn_off VMs running on host */
-  for (const auto& kv : vms_) {
+  for (const auto& [_, vm] : vms_) {
     // call s4u functions to generate the good on_state_change signal, maybe one day this wont be necessary
-    kv.second->get_iface()->shutdown();
-    kv.second->get_iface()->turn_off();
+    vm->get_iface()->shutdown();
+    vm->get_iface()->turn_off();
   }
   for (auto& actor : actor_list_) {
     XBT_DEBUG("Killing Actor %s@%s on behalf of %s which turned off that host.", actor.get_cname(),
@@ -133,8 +138,8 @@ size_t HostImpl::get_actor_count() const
 std::vector<s4u::Disk*> HostImpl::get_disks() const
 {
   std::vector<s4u::Disk*> disks;
-  for (auto const& d : disks_)
-    disks.push_back(d.second->get_iface());
+  for (auto const& [_, d] : disks_)
+    disks.push_back(d->get_iface());
   return disks;
 }
 
@@ -197,8 +202,8 @@ VirtualMachineImpl* HostImpl::get_vm_by_name_or_null(const std::string& name) co
 std::vector<s4u::VirtualMachine*> HostImpl::get_vms() const
 {
   std::vector<s4u::VirtualMachine*> vms;
-  for (const auto& kv : vms_) {
-    vms.push_back(kv.second->get_iface());
+  for (const auto& [_, vm] : vms_) {
+    vms.push_back(vm->get_iface());
   }
   return vms;
 }
@@ -230,12 +235,12 @@ void HostImpl::seal()
   sealed_ = true;
 
   /* seal its disks */
-  for (auto const& disk : disks_)
-    disk.second->seal();
+  for (auto const& [_, disk] : disks_)
+    disk->seal();
 
   /* seal its VMs */
-  for (auto const& vm : vms_)
-    vm.second->seal();
+  for (auto const& [_, vm] : vms_)
+    vm->seal();
 }
 } // namespace resource
 } // namespace kernel