style formatting, some warning cleanup.

This commit is contained in:
wrenczes
2011-05-25 06:59:44 +00:00
parent 3152f69088
commit ffda1d0548
+42 -32
View File
@@ -27,13 +27,7 @@ JSocket::JSocket(string ipAddr)
: state(NOT_AVAILABLE), : state(NOT_AVAILABLE),
mfd(-1) mfd(-1)
{ {
int result = -1; int result;
#ifdef WIN32
SOCKET Desc_Socket_Cliente;
#elif LINUX
int Desc_Socket_Cliente;
#endif
struct hostent *hostentptr; struct hostent *hostentptr;
#ifdef WIN32 #ifdef WIN32
SOCKADDR_IN Adresse_Socket_Server; SOCKADDR_IN Adresse_Socket_Server;
@@ -42,7 +36,8 @@ JSocket::JSocket(string ipAddr)
wVersionRequested=MAKEWORD(1, 1); wVersionRequested=MAKEWORD(1, 1);
result = WSAStartup(wVersionRequested,&wsaData); result = WSAStartup(wVersionRequested,&wsaData);
if(result!=0){ if (result != 0)
{
DebugTrace("WSAStartup\t"); DebugTrace("WSAStartup\t");
return; return;
} }
@@ -61,7 +56,8 @@ JSocket::JSocket(string ipAddr)
#elif LINUX #elif LINUX
hostentptr = gethostbyname(ipAddr.c_str()); hostentptr = gethostbyname(ipAddr.c_str());
#endif #endif
if (hostentptr == NULL) { if (hostentptr == NULL)
{
DebugTrace("ERROR, no such host\n"); DebugTrace("ERROR, no such host\n");
return; return;
} }
@@ -80,7 +76,8 @@ JSocket::JSocket(string ipAddr)
mfd, mfd,
(const struct sockaddr*)&Adresse_Socket_Server, (const struct sockaddr*)&Adresse_Socket_Server,
sizeof(Adresse_Socket_Server)); sizeof(Adresse_Socket_Server));
if (result != 0){ if (result != 0)
{
DebugTrace("client connect failed :" << strerror(errno)); DebugTrace("client connect failed :" << strerror(errno));
state = FATAL_ERROR; state = FATAL_ERROR;
return; return;
@@ -91,17 +88,17 @@ JSocket::JSocket(string ipAddr)
bool JSocket::SetNonBlocking(int sock) bool JSocket::SetNonBlocking(int sock)
{ {
int opts;
#ifdef WIN32 #ifdef WIN32
#elif LINUX #elif LINUX
opts = fcntl(sock,F_GETFL); int opts = fcntl(sock,F_GETFL);
if (opts < 0) { if (opts < 0)
{
perror("fcntl(F_GETFL)"); perror("fcntl(F_GETFL)");
return false; return false;
} }
opts = (opts | O_NONBLOCK); opts = (opts | O_NONBLOCK);
if (fcntl(sock,F_SETFL,opts) < 0) { if (fcntl(sock,F_SETFL,opts) < 0)
{
perror("fcntl(F_SETFL)"); perror("fcntl(F_SETFL)");
return false; return false;
} }
@@ -122,7 +119,8 @@ JSocket::JSocket()
wVersionRequested = MAKEWORD(1,1); wVersionRequested = MAKEWORD(1,1);
result = WSAStartup(wVersionRequested,&wsaData); result = WSAStartup(wVersionRequested,&wsaData);
if(result!=0){ if (result != 0)
{
DebugTrace("WSAStartup\t"); DebugTrace("WSAStartup\t");
return; return;
} }
@@ -155,14 +153,16 @@ JSocket::JSocket()
result = ::bind(mfd, result = ::bind(mfd,
(struct sockaddr*)&Adresse_Socket_Connection, (struct sockaddr*)&Adresse_Socket_Connection,
sizeof(Adresse_Socket_Connection)); sizeof(Adresse_Socket_Connection));
if(result!=0){ if (result != 0)
{
state = FATAL_ERROR; state = FATAL_ERROR;
DebugTrace("bind error:" << strerror(errno)); DebugTrace("bind error:" << strerror(errno));
return; return;
} }
result = listen(mfd,1); result = listen(mfd,1);
if(result!=0){ if(result != 0)
{
state = FATAL_ERROR; state = FATAL_ERROR;
DebugTrace("listen error:" << strerror(errno)); DebugTrace("listen error:" << strerror(errno));
return; return;
@@ -177,7 +177,8 @@ JSocket::JSocket(int fd)
{ {
} }
JSocket::~JSocket(){ JSocket::~JSocket()
{
Disconnect(); Disconnect();
#ifdef WIN32 #ifdef WIN32
WSACleanup(); WSACleanup();
@@ -187,7 +188,8 @@ JSocket::~JSocket(){
void JSocket::Disconnect() void JSocket::Disconnect()
{ {
state = JSocket::DISCONNECTED; state = JSocket::DISCONNECTED;
if(mfd) { if(mfd)
{
#ifdef WIN32 #ifdef WIN32
closesocket(mfd); closesocket(mfd);
#elif LINUX #elif LINUX
@@ -207,7 +209,10 @@ JSocket* JSocket::Accept()
socklen_t Longueur_Adresse; socklen_t Longueur_Adresse;
#endif //WINDOWS #endif //WINDOWS
while(mfd) { JSocket* socket = NULL;
while (mfd)
{
fd_set set; fd_set set;
FD_ZERO(&set); FD_ZERO(&set);
FD_SET(mfd, &set); FD_SET(mfd, &set);
@@ -216,7 +221,8 @@ JSocket* JSocket::Accept()
tv.tv_usec = 1000 * 100; tv.tv_usec = 1000 * 100;
int result = select(mfd+1, &set, NULL, NULL, &tv); int result = select(mfd+1, &set, NULL, NULL, &tv);
if( result > 0 && FD_ISSET(mfd, &set) ) { if (result > 0 && FD_ISSET(mfd, &set))
{
Longueur_Adresse = sizeof(Adresse_Socket_Cliente); Longueur_Adresse = sizeof(Adresse_Socket_Cliente);
int val = accept( int val = accept(
@@ -225,14 +231,16 @@ JSocket* JSocket::Accept()
&Longueur_Adresse); &Longueur_Adresse);
DebugTrace("connection on client port "<< ntohs(Adresse_Socket_Cliente.sin_port)); DebugTrace("connection on client port "<< ntohs(Adresse_Socket_Cliente.sin_port));
if(val >=0 ) { if (val >= 0)
{
state = CONNECTED; state = CONNECTED;
return new JSocket(val); socket = new JSocket(val);
} else { }
return NULL; break;
}
} }
} }
return socket;
} }
int JSocket::Read(char* buff, int size) int JSocket::Read(char* buff, int size)
@@ -273,7 +281,8 @@ int JSocket::Read(char* buff, int size)
int JSocket::Write(char* buff, int size) int JSocket::Write(char* buff, int size)
{ {
int size1 = size; int size1 = size;
while (size > 0 && state == CONNECTED) { while (size > 0 && state == CONNECTED)
{
fd_set set; fd_set set;
FD_ZERO(&set); FD_ZERO(&set);
FD_SET(mfd, &set); FD_SET(mfd, &set);
@@ -285,16 +294,17 @@ int JSocket::Write(char* buff, int size)
if( result > 0 && FD_ISSET(mfd, &set)) if( result > 0 && FD_ISSET(mfd, &set))
{ {
int len = send(mfd, buff, size, 0); int len = send(mfd, buff, size, 0);
if (len < 0) { if (len < 0)
{
return -1; return -1;
} }
size -= len; size -= len;
buff += len; buff += len;
} else if (result < 0) { }
else if (result < 0)
{
return -1; return -1;
} }
} }
return size1 - size; return size1 - size;
} }