diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2016-06-16 08:08:10 +0000 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2016-06-16 10:51:40 +0000 |
commit | 7cb778378fdf92331882704569d72e3dadee2c22 (patch) | |
tree | 6129dcf29a70d7b1b5878652c1ede8b5076dd008 /fastos | |
parent | b60d8747e307e2868be7ef53d86fbce1c2cbc4d6 (diff) |
GC some more obscuring defines
Diffstat (limited to 'fastos')
-rw-r--r-- | fastos/src/vespa/fastos/autoconf.h | 1 | ||||
-rw-r--r-- | fastos/src/vespa/fastos/serversocket.cpp | 4 | ||||
-rw-r--r-- | fastos/src/vespa/fastos/socket.cpp | 27 | ||||
-rw-r--r-- | fastos/src/vespa/fastos/types.h | 9 |
4 files changed, 10 insertions, 31 deletions
diff --git a/fastos/src/vespa/fastos/autoconf.h b/fastos/src/vespa/fastos/autoconf.h index 955f16fd84d..d4a70654f71 100644 --- a/fastos/src/vespa/fastos/autoconf.h +++ b/fastos/src/vespa/fastos/autoconf.h @@ -13,4 +13,3 @@ #define FASTOS_HAVE_INT8_T 1 #define FASTOS_HAVE_UINT8_T 1 #define FASTOS_HAVE_INT64_T 1 -#define FASTOS_HAVE_SOCKLEN_T 1 diff --git a/fastos/src/vespa/fastos/serversocket.cpp b/fastos/src/vespa/fastos/serversocket.cpp index eb4a737683a..15f52e83bc1 100644 --- a/fastos/src/vespa/fastos/serversocket.cpp +++ b/fastos/src/vespa/fastos/serversocket.cpp @@ -43,7 +43,7 @@ FastOS_SocketInterface *FastOS_ServerSocket::Accept() int handlerSocketHandle; struct sockaddr_storage clientAddress; - FastOS_SocketLen clientAddressLength = sizeof(clientAddress); + socklen_t clientAddressLength = sizeof(clientAddress); memset(&clientAddress, 0, sizeof(clientAddress)); @@ -71,7 +71,7 @@ FastOS_Socket *FastOS_ServerSocket::AcceptPlain() int handlerSocketHandle; struct sockaddr_storage clientAddress; - FastOS_SocketLen clientAddressLength = sizeof(clientAddress); + socklen_t clientAddressLength = sizeof(clientAddress); memset(&clientAddress, 0, sizeof(clientAddress)); diff --git a/fastos/src/vespa/fastos/socket.cpp b/fastos/src/vespa/fastos/socket.cpp index d97fe98a785..4e30a9216ea 100644 --- a/fastos/src/vespa/fastos/socket.cpp +++ b/fastos/src/vespa/fastos/socket.cpp @@ -3,8 +3,6 @@ #include <vespa/fastos/socket.h> #include <sstream> -typedef caddr_t FastOS_SockOptValP; - FastOS_SocketInterface::FastOS_SocketInterface() : _readEventEnabled(false), _writeEventEnabled(false), @@ -158,9 +156,7 @@ bool FastOS_SocketInterface::SetSoLinger( bool doLinger, int seconds ) lingerTime.l_linger = seconds; if (CreateIfNoSocketYet()) { - rc = (0 == setsockopt(_socketHandle, SOL_SOCKET, SO_LINGER, - reinterpret_cast<FastOS_SockOptValP>(&lingerTime), - sizeof(lingerTime))); + rc = (0 == setsockopt(_socketHandle, SOL_SOCKET, SO_LINGER, &lingerTime, sizeof(lingerTime))); } return rc; @@ -173,9 +169,7 @@ FastOS_SocketInterface::SetNoDelay(bool noDelay) int noDelayInt = noDelay ? 1 : 0; if (CreateIfNoSocketYet()) { - rc = (setsockopt(_socketHandle, IPPROTO_TCP, TCP_NODELAY, - reinterpret_cast<FastOS_SockOptValP>(&noDelayInt), - sizeof(noDelayInt)) == 0); + rc = (setsockopt(_socketHandle, IPPROTO_TCP, TCP_NODELAY, &noDelayInt, sizeof(noDelayInt)) == 0); } return rc; } @@ -189,10 +183,9 @@ int FastOS_SocketInterface::GetSoError () int lastError = FastOS_Socket::GetLastError(); int soError = 0; - FastOS_SocketLen soErrorLen = sizeof(soError); + socklen_t soErrorLen = sizeof(soError); - if (getsockopt(_socketHandle, SOL_SOCKET, SO_ERROR, - reinterpret_cast<FastOS_SockOptValP>(&soError), &soErrorLen) != 0) { + if (getsockopt(_socketHandle, SOL_SOCKET, SO_ERROR, &soError, &soErrorLen) != 0) { return lastError; } @@ -208,9 +201,7 @@ bool FastOS_SocketInterface::SetSoIntOpt (int option, int value) bool rc=false; if (CreateIfNoSocketYet()) { - rc = (0 == setsockopt(_socketHandle, SOL_SOCKET, option, - reinterpret_cast<FastOS_SockOptValP>(&value), - sizeof(value))); + rc = (0 == setsockopt(_socketHandle, SOL_SOCKET, option, &value, sizeof(value))); } return rc; @@ -221,11 +212,9 @@ bool FastOS_SocketInterface::GetSoIntOpt(int option, int &value) bool rc=false; if (CreateIfNoSocketYet()) { - FastOS_SocketLen len = sizeof(value); + socklen_t len = sizeof(value); - int retval = getsockopt(_socketHandle, SOL_SOCKET, option, - reinterpret_cast<FastOS_SockOptValP>(&value), - &len); + int retval = getsockopt(_socketHandle, SOL_SOCKET, option, &value, &len); if (len != sizeof(value)) { // FIX! - What about GetLastError() in this case? @@ -264,7 +253,7 @@ int FastOS_SocketInterface::GetLocalPort () { int result = -1; sockaddr_storage addr; - FastOS_SocketLen len = sizeof(addr); + socklen_t len = sizeof(addr); if(getsockname(_socketHandle, reinterpret_cast<struct sockaddr *>(&addr), &len) == 0) { if ((addr.ss_family == AF_INET) && (len == sizeof(sockaddr_in))) { const sockaddr_in *my_addr = reinterpret_cast<const sockaddr_in *>(&addr); diff --git a/fastos/src/vespa/fastos/types.h b/fastos/src/vespa/fastos/types.h index 39eec1bbdbd..36a7ba7c926 100644 --- a/fastos/src/vespa/fastos/types.h +++ b/fastos/src/vespa/fastos/types.h @@ -97,15 +97,6 @@ #undef __USE_GNU /* for O_DIRECT define */ #endif -#ifndef FASTOS_AUTOCONF -#if defined(FASTOS_HAVE_SOCKLEN_T) -typedef socklen_t FastOS_SocketLen; -#else -typedef int FastOS_SocketLen; -#endif -#endif /* FASTOS_AUTOCONF */ - - #include <stdio.h> #include <stdlib.h> |