Clean-up of the code related to attaching/detaching engines to sessions.

Session base class now handles the engine events exclusively. It notifies
derived session types using dedicated "attached" and "detached" events.

Couple of bugs was fixed along the way.

Signed-off-by: Martin Sustrik <sustrik@250bpm.com>

1 files changed, 4 insertions, 5 deletions

diff --git a/src/connect_session.cpp b/src/connect_session.cpp
index 833b551..4d8e322 100644
--- a/src/connect_session.cpp
+++ b/src/connect_session.cpp
@@ -105,14 +105,13 @@ void zmq::connect_session_t::start_connecting ()
     zmq_assert (false);
 }
 
-void zmq::connect_session_t::detached ()
+void zmq::connect_session_t::attached (const blob_t &peer_identity_)
 {
-    //  Clean up the mess left over by the failed connection.
-    clean_pipes ();
+}
 
+void zmq::connect_session_t::detached ()
+{
     //  Reconnect.
     start_connecting ();
-
-    session_t::detached ();
 }