Error: rename SESSION_CLOSED to SESSION_NOT_OPEN for clarity

asyncmsg/Error.h

@@ -11,7 +11,7 @@ namespace asyncmsg
 enum class Error:
     int
 {
-  SESSION_CLOSED,
+  SESSION_NOT_OPEN,
   SESSION_NOT_CLOSED,
   RESERVED_MESSAGE_TYPE,
   UNEXPECTED_MESSAGE_TYPE,

src/Error.cxx

@@ -19,8 +19,8 @@ ErrorCategory ErrorCategory::s_instance;
 
 std::string ErrorCategory::message(int value) const
 {
-  if (value == static_cast<int>(Error::SESSION_CLOSED)) {
-    return "The session is closed";
+  if (value == static_cast<int>(Error::SESSION_NOT_OPEN)) {
+    return "The session is not open";
   }
   if (value == static_cast<int>(Error::SESSION_NOT_CLOSED)) {
     return "The session is not closed";

src/Session.cxx

@@ -137,7 +137,7 @@ void Session::asyncClose()
           // use post to avoid stack overflows if the user calls asyncClose again from onCloseError()
           m_strand.post(
               [this, self] () {
-                onCloseError(Error::SESSION_CLOSED);
+                onCloseError(Error::SESSION_NOT_OPEN);
               });
         }
         else {
@@ -169,7 +169,7 @@ void Session::asyncReceive()
           // onReceiveError()
           m_strand.post(
               [this, self] () {
-                onReceiveError(Error::SESSION_CLOSED, std::unique_ptr<InputMessage>());
+                onReceiveError(Error::SESSION_NOT_OPEN, std::unique_ptr<InputMessage>());
               });
         }
         else {
@@ -251,7 +251,7 @@ void Session::asyncSend(std::unique_ptr<const OutputMessage> message)
           // use post to avoid stack overflows if the user calls asyncSend again from onSendError()
           m_strand.post(
               [this, self, bareMessage] () {
-                onSendError(Error::SESSION_CLOSED, std::unique_ptr<const OutputMessage>(bareMessage));
+                onSendError(Error::SESSION_NOT_OPEN, std::unique_ptr<const OutputMessage>(bareMessage));
               });
         }
         else if (bareMessage->typeId() >= RESERVED_MESSAGE_TYPES_BASE) {

src/UDPSession.cxx

@@ -57,7 +57,7 @@ void UDPSession::asyncReceive()
           // onReceiveError()
           m_strand.post(
               [this, self] () {
-                onReceiveError(Error::SESSION_CLOSED, std::unique_ptr<InputMessage>());
+                onReceiveError(Error::SESSION_NOT_OPEN, std::unique_ptr<InputMessage>());
               });
         }
         else {
@@ -129,7 +129,7 @@ void UDPSession::asyncSend(std::unique_ptr<const OutputMessage> message)
           // use post to avoid stack overflows if the user calls asyncSend again from onSendError()
           m_strand.post(
               [this, self, bareMessage] () {
-                onSendError(Error::SESSION_CLOSED, std::unique_ptr<const OutputMessage>(bareMessage));
+                onSendError(Error::SESSION_NOT_OPEN, std::unique_ptr<const OutputMessage>(bareMessage));
               });
         }
         else if (bareMessage->typeId() >= RESERVED_MESSAGE_TYPES_BASE) {