X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fstreamsocket.cpp;h=8a262454eca173dfcc6ef93f0fcd0d45c10582f7;hb=21c596567e3b6fd794ed6af73d304ce2bc70e58f;hp=cd810d5253e14e2990d204c4af12096f5be7278b;hpb=4696a776e2875697ea0d6a733609dd9c37918971;p=libs%2Fnet.git

diff --git a/source/streamsocket.cpp b/source/streamsocket.cpp
index cd810d5..8a26245 100644
--- a/source/streamsocket.cpp
+++ b/source/streamsocket.cpp
@@ -3,98 +3,51 @@
 #endif
 #include <cerrno>
 #include <msp/core/systemerror.h>
+#include <msp/io/handle_private.h>
 #include <msp/io/poll.h>
 #include <msp/strings/format.h>
+#include "sockaddr_private.h"
+#include "socket_private.h"
 #include "streamsocket.h"
 
 namespace Msp {
 namespace Net {
 
-StreamSocket::StreamSocket(SocketHandle h, const SockAddr &paddr):
-	Socket(h, paddr),
-	connecting(false)
+StreamSocket::StreamSocket(const Private &p, const SockAddr &paddr):
+	ClientSocket(p, paddr)
 {
 #ifdef WIN32
-	WSAEventSelect(handle, event, FD_READ|FD_CLOSE);
+	WSAEventSelect(priv->handle, *priv->event, FD_READ|FD_CLOSE);
 #endif
 	set_events(IO::P_INPUT);
 }
 
 StreamSocket::StreamSocket(Family af, int proto):
-	Socket(af, SOCK_STREAM, proto),
-	connecting(false)
+	ClientSocket(af, SOCK_STREAM, proto)
 { }
 
-int StreamSocket::poll_connect(const Time::TimeDelta &timeout)
+bool StreamSocket::connect(const SockAddr &addr)
 {
-	check_state(false);
-	if(!connecting)
-		throw bad_socket_state("not connecting");
-
-	int res = poll(*this, IO::P_OUTPUT, timeout);
-	if(res==-1)
-#ifdef WIN32
-		throw system_error("poll", WSAGetLastError());
-#else
-		throw system_error("poll");
-#endif
-	else if(res>0)
-	{
-		connecting = false;
-
-		int err;
-		socklen_t len = sizeof(int);
-		get_option(SOL_SOCKET, SO_ERROR, &err, &len);
-
-		if(err!=0)
-		{
-			set_events(IO::P_NONE);
-#ifdef WIN32
-			throw system_error("connect", WSAGetLastError());
-#else
-			throw system_error("connect");
-#endif
-		}
-
-#ifdef WIN32
-		WSAEventSelect(handle, event, FD_READ|FD_CLOSE);
-#endif
-		set_events(IO::P_INPUT);
-
-		connected = true;
-
-		return 0;
-	}
-
-	return 1;
-}
-
-int StreamSocket::connect(const SockAddr &addr)
-{
-	check_state(false);
-
 	if(connected)
 		throw bad_socket_state("already connected");
 
-	sockaddr_storage sa;
-	socklen_t size = addr.fill_sockaddr(sa);
+	SockAddr::SysAddr sa = addr.to_sys();
 
+	int err = ::connect(priv->handle, reinterpret_cast<sockaddr *>(&sa.addr), sa.size);
 #ifdef WIN32
-	int err = WSAConnect(handle, reinterpret_cast<sockaddr *>(&sa), size, 0, 0, 0, 0);
 	if(err==SOCKET_ERROR)
 	{
 		int err_code = WSAGetLastError();
 		if(err_code==WSAEWOULDBLOCK)
 		{
 			connecting = true;
-			WSAEventSelect(handle, event, FD_CONNECT);
+			WSAEventSelect(priv->handle, *priv->event, FD_CONNECT);
 			set_events(IO::P_OUTPUT);
 		}
 		else
 			throw system_error("connect", err_code);
 	}
 #else
-	int err = ::connect(handle, reinterpret_cast<sockaddr *>(&sa), size);
 	if(err==-1)
 	{
 		if(errno==EINPROGRESS)
@@ -111,9 +64,9 @@ int StreamSocket::connect(const SockAddr &addr)
 	peer_addr = addr.copy();
 
 	delete local_addr;
-	size = sizeof(sockaddr_storage);
-	getsockname(handle, reinterpret_cast<sockaddr *>(&sa), &size);
-	local_addr = SockAddr::create(sa);
+	SockAddr::SysAddr lsa;
+	getsockname(priv->handle, reinterpret_cast<sockaddr *>(&lsa.addr), &lsa.size);
+	local_addr = SockAddr::new_from_sys(lsa);
 
 	if(err==0)
 	{
@@ -122,7 +75,42 @@ int StreamSocket::connect(const SockAddr &addr)
 		signal_connect_finished.emit(0);
 	}
 
-	return (err==0)?0:1;
+	return connected;
+}
+
+bool StreamSocket::poll_connect(const Time::TimeDelta &timeout)
+{
+	if(!connecting)
+		return false;
+
+	IO::PollEvent res = poll(*this, IO::P_OUTPUT, timeout);
+	if(res&IO::P_OUTPUT)
+	{
+		connecting = false;
+
+		int err;
+		socklen_t len = sizeof(int);
+		get_option(SOL_SOCKET, SO_ERROR, &err, &len);
+
+		if(err!=0)
+		{
+			set_events(IO::P_NONE);
+#ifdef WIN32
+			throw system_error("connect", WSAGetLastError());
+#else
+			throw system_error("connect");
+#endif
+		}
+
+#ifdef WIN32
+		WSAEventSelect(priv->handle, *priv->event, FD_READ|FD_CLOSE);
+#endif
+		set_events(IO::P_INPUT);
+
+		connected = true;
+	}
+
+	return connected;
 }
 
 void StreamSocket::on_event(IO::PollEvent ev)
@@ -135,7 +123,13 @@ void StreamSocket::on_event(IO::PollEvent ev)
 
 		connecting = false;
 		connected = (err==0);
-		signal_connect_finished.emit(err);
+		if(err)
+		{
+			system_error exc("connect", err);
+			signal_connect_finished.emit(&exc);
+		}
+		else
+			signal_connect_finished.emit(0);
 
 		if(err!=0)
 		{
@@ -144,7 +138,7 @@ void StreamSocket::on_event(IO::PollEvent ev)
 		}
 
 #ifdef WIN32
-		WSAEventSelect(handle, event, FD_READ|FD_CLOSE);
+		WSAEventSelect(priv->handle, *priv->event, FD_READ|FD_CLOSE);
 #endif
 		set_events((err==0) ? IO::P_INPUT : IO::P_NONE);
 	}