{
lastErrorMsg=e.what();
LOGSTREAM(debug,"tcp connect error: " << lastErrorMsg);
+ // FIXME: Don't call virtual function in constructor. Currently not dangerous but bad design.
close();
}
{
lastErrorMsg=e.what();
LOGSTREAM(debug,"unix connect error: " << lastErrorMsg);
- // FIXME: Calls virtual function close in constructor
+ // FIXME: Don't call virtual function in constructor. Currently not dangerous
close();
}
do_callbacks(new_connection);
}
+/**
+ * Destructor. Closes an open connection.
+ */
+socket_client_connection::~socket_client_connection()
+{
+ // Destructor of socket_handler will close the socket!
+}
+
+
/// establish a connection via tcp
void socket_client_connection::tcp_connect(int max_retries)
{