From 7a468f358be95632b9501a91f18a07555dedaaf3 Mon Sep 17 00:00:00 2001 From: John Spray Date: Thu, 25 Sep 2014 17:01:10 +0100 Subject: [PATCH] msg: allow calling dtor immediately after ctor Asserting on reaper_stop only made sense if the messenger had ever been started: as it stood, one couldn't create and destroy a messenger without also starting and stopping it. Signed-off-by: John Spray --- src/msg/SimpleMessenger.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/msg/SimpleMessenger.cc b/src/msg/SimpleMessenger.cc index f1f052bce92a..cf6bc73711ba 100644 --- a/src/msg/SimpleMessenger.cc +++ b/src/msg/SimpleMessenger.cc @@ -65,7 +65,7 @@ SimpleMessenger::~SimpleMessenger() { assert(!did_bind); // either we didn't bind or we shut down the Accepter assert(rank_pipe.empty()); // we don't have any running Pipes. - assert(reaper_stop && !reaper_started); // the reaper thread is stopped + assert(!reaper_started); // the reaper thread is stopped } void SimpleMessenger::ready() -- 2.47.3