Code cleanup

src/session_base.cpp

@@ -38,7 +38,6 @@ zmq::session_base_t *zmq::session_base_t::create (class io_thread_t *io_thread_,
     bool active_, class socket_base_t *socket_, const options_t &options_,
     address_t *addr_)
 {
-	
     session_base_t *s = NULL;
     switch (options_.type) {
     case ZMQ_REQ:
@@ -228,8 +227,10 @@ void zmq::session_base_t::pipe_terminated (pipe_t *pipe_)
     //  If we are waiting for pending messages to be sent, at this point
     //  we are sure that there will be no more messages and we can proceed
     //  with termination safely.
-    if (pending && !pipe && !zap_pipe && terminating_pipes.empty ())
-        proceed_with_term ();
+    if (pending && !pipe && !zap_pipe && terminating_pipes.empty ()) {
+        pending = false;
+        own_t::process_term (0);
+    }
 }
 
 void zmq::session_base_t::read_activated (pipe_t *pipe_)
@@ -354,7 +355,7 @@ void zmq::session_base_t::process_attach (i_engine *engine_)
         //  Remember the local end of the pipe.
         zmq_assert (!pipe);
         pipe = pipes [0];
-        // Store engine assoc_fd for lilnking pipe to fd 
+        // Store engine assoc_fd for linking pipe to fd
         pipe->assoc_fd = engine_->get_assoc_fd ();
         pipes [1]->assoc_fd = pipe->assoc_fd;
         //  Ask socket to plug into the remote end of the pipe.
@@ -409,8 +410,8 @@ void zmq::session_base_t::process_term (int linger_)
     //  If the termination of the pipe happens before the term command is
     //  delivered there's nothing much to do. We can proceed with the
     //  standard termination immediately.
-    if (!pipe && !zap_pipe) {
-        proceed_with_term ();
+    if (!pipe && !zap_pipe && terminating_pipes.empty ()) {
+        own_t::process_term (0);
         return;
     }
 
@@ -440,15 +441,6 @@ void zmq::session_base_t::process_term (int linger_)
         zap_pipe->terminate (false);
 }
 
-void zmq::session_base_t::proceed_with_term ()
-{
-    //  The pending phase has just ended.
-    pending = false;
-
-    //  Continue with standard termination.
-    own_t::process_term (0);
-}
-
 void zmq::session_base_t::timer_event (int id_)
 {
     //  Linger period expired. We can proceed with termination even though
@@ -580,8 +572,7 @@ void zmq::session_base_t::start_connecting (bool wait_)
 #endif
 
 #ifdef ZMQ_HAVE_NORM
-    if (addr->protocol == "norm")
-    {
+    if (addr->protocol == "norm") {
         //  At this point we'll create message pipes to the session straight
         //  away. There's no point in delaying it as no concept of 'connect'
         //  exists with NORM anyway.

src/session_base.hpp

@@ -112,12 +112,9 @@ namespace zmq
         //  Call this function when engine disconnect to get rid of leftovers.
         void clean_pipes ();
 
-        //  Call this function to move on with the delayed process_term.
-        void proceed_with_term ();
-
         //  If true, this session (re)connects to the peer. Otherwise, it's
         //  a transient session created by the listener.
-        bool active;
+        const bool active;
 
         //  Pipe connecting the session to its socket.
         zmq::pipe_t *pipe;

src/xpub.cpp

@@ -27,10 +27,10 @@
 zmq::xpub_t::xpub_t (class ctx_t *parent_, uint32_t tid_, int sid_) :
     socket_base_t (parent_, tid_, sid_),
     verbose (false),
-    more (false)
+    more (false),
+    lossy (true)
 {
     options.type = ZMQ_XPUB;
-    lossy = true;
 }
 
 zmq::xpub_t::~xpub_t ()