firing on_this_completion breaks a lot of tests and I don't see how to fix this ...

diff --git a/src/kernel/activity/CommImpl.cpp b/src/kernel/activity/CommImpl.cpp
index ad1c7fe..dbf9510 100644 (file)
--- a/src/kernel/activity/CommImpl.cpp
+++ b/src/kernel/activity/CommImpl.cpp
@@ -470,9 +470,9 @@ void CommImpl::finish()
   XBT_DEBUG("CommImpl::finish() comm %p, state %s, src_proc %p, dst_proc %p, detached: %d", this, get_state_str(),
             src_actor_.get(), dst_actor_.get(), detached_);
 
-  auto* piface = static_cast<const s4u::Comm*>(this->get_iface());
-  s4u::Comm::on_completion(*piface);
-  piface->on_this_completion(*piface);
+  const s4u::Comm& piface = static_cast<const s4u::Comm&>(*this->get_iface());
+  s4u::Comm::on_completion(piface);
+  //piface.on_this_completion(piface);
 
   /* Update synchro state */
   if (src_timeout_ && src_timeout_->get_state() == resource::Action::State::FINISHED)