commit 04b303819eace72df588da0db951ac6ed2cb0b52
author Tom Joseph <tomjoseph@in.ibm.com> Mon Apr 03 01:06:51 2017 +0530
committer Patrick Williams <patrick@stwcx.xyz> Tue Apr 25 19:12:26 2017 +0000
tree 5fd993a292ff1c2aa03a10ec179a318739e9bc1b
parent 63d3e49c095e3d06ffcae39e92d5020e348e0a64

Throw exception instead of returning error code from send function.

Change-Id: I87d2805dfb7f8757c0366600d8937023edddf54a
Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>

message_handler.cpp

diff --git a/message_handler.cpp b/message_handler.cpp
index ef9eb06..e66e39c 100644
--- a/message_handler.cpp
+++ b/message_handler.cpp
@@ -163,7 +163,7 @@
     return command;
 }
 
-int Handler::send(Message& outMessage)
+void Handler::send(Message& outMessage)
 {
     auto session = (std::get<session::Manager&>(singletonPool).getSession(
                     sessionID)).lock();
@@ -171,15 +171,12 @@
     // Flatten the packet
     auto packet = parser::flatten(outMessage, sessionHeader, *session);
 
-    // Read the packet
+    // Write the packet
     auto writeStatus = channel->write(packet);
     if (writeStatus < 0)
     {
-        std::cerr << "E> Error in writing : " << std::hex << writeStatus
-                  << "\n";
+        throw std::runtime_error("Error in writing to socket");
     }
-
-    return writeStatus;
 }
 
 void Handler::setChannelInSession() const