Move the basic socket implementation into the SocketHandler class

(that is, the read/write/connect/etc)
Because this code is actually common for both the IrcClient and
XmppComponent class. These two classes have to implement some higher level
callbacks (parsing the data provided in the buffers, doing stuff when the
connection is done) instead of doing the read/write/connect low level
things.

2 files changed, 1 insertions, 126 deletions

diff --git a/src/libirc/irc_client.cpp b/src/libirc/irc_client.cpp
index 2780b3c..a427026 100644
--- a/src/libirc/irc_client.cpp
+++ b/src/libirc/irc_client.cpp
@@ -1,13 +1,5 @@
 #include <libirc/irc_client.hpp>
 #include <libirc/irc_message.hpp>
-#include <network/poller.hpp>
-#include <utils/scopeguard.hpp>
-
-#include <sys/types.h>
-#include <sys/socket.h>
-#include <cstring>
-#include <netdb.h>
-#include <unistd.h>
 
 #include <iostream>
 #include <stdexcept>
@@ -15,8 +7,6 @@
 IrcClient::IrcClient()
 {
   std::cout << "IrcClient()" << std::endl;
-  if ((this->socket = ::socket(AF_INET, SOCK_STREAM, 0)) == -1)
-    throw std::runtime_error("Could not create socket");
 }
 
 IrcClient::~IrcClient()
@@ -24,80 +14,6 @@ IrcClient::~IrcClient()
   std::cout << "~IrcClient()" << std::endl;
 }
 
-void IrcClient::on_recv()
-{
-  char buf[4096];
-
-  ssize_t size = ::recv(this->socket, buf, 4096, 0);
-  if (0 == size)
-    this->on_connection_close();
-  else if (-1 == static_cast<ssize_t>(size))
-    throw std::runtime_error("Error reading from socket");
-  else
-    {
-      this->in_buf += std::string(buf, size);
-      this->parse_in_buffer();
-    }
-}
-
-void IrcClient::on_send()
-{
-  const ssize_t res = ::send(this->socket, this->out_buf.data(), this->out_buf.size(), 0);
-  if (res == -1)
-    {
-      perror("send");
-      this->close();
-    }
-  else
-    {
-      this->out_buf = this->out_buf.substr(res, std::string::npos);
-      if (this->out_buf.empty())
-        this->poller->stop_watching_send_events(this);
-    }
-}
-
-socket_t IrcClient::get_socket() const
-{
-  return this->socket;
-}
-
-void IrcClient::connect(const std::string& address, const std::string& port)
-{
-  std::cout << "Trying to connect to " << address << ":" << port << std::endl;
-  struct addrinfo hints;
-  memset(&hints, 0, sizeof(struct addrinfo));
-  hints.ai_flags = 0;
-  hints.ai_family = AF_INET;
-  hints.ai_socktype = SOCK_STREAM;
-  hints.ai_protocol = 0;
-
-  struct addrinfo* addr_res;
-  const int res = ::getaddrinfo(address.c_str(), port.c_str(), &hints, &addr_res);
-  // Make sure the alloced structure is always freed at the end of the
-  // function
-  utils::ScopeGuard sg([&addr_res](){ freeaddrinfo(addr_res); });
-
-  if (res != 0)
-    {
-      perror("getaddrinfo");
-      throw std::runtime_error("getaddrinfo failed");
-    }
-  for (struct addrinfo* rp = addr_res; rp; rp = rp->ai_next)
-    {
-      std::cout << "One result" << std::endl;
-      if (::connect(this->socket, rp->ai_addr, rp->ai_addrlen) == 0)
-        {
-          std::cout << "Connection success." << std::endl;
-          this->on_connected();
-          return ;
-        }
-      std::cout << "Connection failed:" << std::endl;
-      perror("connect");
-    }
-  std::cout << "All connection attempts failed." << std::endl;
-  this->close();
-}
-
 void IrcClient::on_connected()
 {
 }
@@ -108,12 +24,6 @@ void IrcClient::on_connection_close()
   this->close();
 }
 
-void IrcClient::close()
-{
-  this->poller->remove_socket_handler(this->get_socket());
-  ::close(this->socket);
-}
-
 void IrcClient::parse_in_buffer()
 {
   while (true)
@@ -143,11 +53,7 @@ void IrcClient::send_message(IrcMessage&& message)
       res += " " + arg;
     }
   res += "\r\n";
-  this->out_buf += res;
-  if (!this->out_buf.empty())
-    {
-      this->poller->watch_send_events(this);
-    }
+  this->send_data(std::move(res));
 }
 
 void IrcClient::send_user_command(const std::string& username, const std::string& realname)
diff --git a/src/libirc/irc_client.hpp b/src/libirc/irc_client.hpp
index d1ecbd5..9778876 100644
--- a/src/libirc/irc_client.hpp
+++ b/src/libirc/irc_client.hpp
@@ -19,30 +19,12 @@ public:
   explicit IrcClient();
   ~IrcClient();
   /**
-   * We read the data, try to parse it and generate some event if
-   * one or more full message is available.
-   */
-  void on_recv();
-  /**
-   * Just write as much data as possible on the socket.
-   */
-  void on_send();
-  socket_t get_socket() const;
-  /**
-   * Connect to the remote server
-   */
-  void connect(const std::string& address, const std::string& port);
-  /**
    * Called when successfully connected to the server
    */
   void on_connected();
   /**
    * Close the connection, remove us from the poller
    */
-  void close();
-  /**
-   * Called when we detect an orderly close by the remote endpoint.
-   */
   void on_connection_close();
   /**
    * Parse the data we have received so far and try to get one or more
@@ -69,19 +51,6 @@ public:
   void send_join_command(const std::string& chan_name);
 
 private:
-  socket_t socket;
-  /**
-   * Where data read from the socket is added, until we can parse a whole
-   * IRC message, the used data is then removed from that buffer.
-   *
-   * TODO: something more efficient than a string.
-   */
-  std::string in_buf;
-  /**
-   * Where data is added, when we want to send something to the client.
-   */
-  std::string out_buf;
-
   IrcClient(const IrcClient&) = delete;
   IrcClient(IrcClient&&) = delete;
   IrcClient& operator=(const IrcClient&) = delete;