Replaced:

if (state_ == finished)
    xbt_die("Don't call test on a finished comm.");

by:

if (state_ == finished)
    return true;

Since testing a finished comm should be a feature, not a bug

diff --git a/src/s4u/s4u_comm.cpp b/src/s4u/s4u_comm.cpp
index 7729d3e..17a66b6 100644 (file)
--- a/src/s4u/s4u_comm.cpp
+++ b/src/s4u/s4u_comm.cpp
@@ -179,8 +179,9 @@ void Comm::cancel()
 bool Comm::test() {
   xbt_assert(state_ == inited || state_ == started || state_ == finished);
 
-  if (state_ == finished)
-    xbt_die("Don't call test on a finished comm.");
+  if (state_ == finished) {
+    return true;
+  }
 
   if (state_ == inited) {
     this->start();