nerdcruft/libcruft-util
1
0
Fork 0
Code Issues Releases Activity

socket: explain and relax type casting requirement

Browse Source 
The recv/send functions (and the assertions) were only tested under
32bit systems. They need relaxing for 64bit.
This commit is contained in:
Danny Robson 2014-12-05 13:21:14 +11:00
parent b8ba95f365
commit 976a4b7330
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
net/socket.cpp
View File

@ -52,8 +52,12 @@ socket_domain<D>::socket_domain (socket_t _fd):
// TODO: Make this not retarded. Fucking Windows. // TODO: Make this not retarded. Fucking Windows.
#define dup(X) (X) #define dup(X) (X)
static_assert(sizeof(int) == sizeof(ssize_t), "int != ssize_t"); // Winsock has incorrect return and parameter types (according to POSIX)
static_assert(sizeof(int) == sizeof( size_t), "int != size_t"); // so we need some wrappers around the system implementations that casts
// to expected types. Mainly int vs ssize_t returns, and int vs size_t
// parameters.
static_assert(sizeof(int) <= sizeof(ssize_t), "int != ssize_t");
static_assert(sizeof(int) <= sizeof( size_t), "int != size_t");
ssize_t recv(socket_t _socket, void *_buf, size_t _len, int _flags) ssize_t recv(socket_t _socket, void *_buf, size_t _len, int _flags)
{ return (ssize_t)::recv(_socket, (char*)_buf, (int)_len, _flags); } { return (ssize_t)::recv(_socket, (char*)_buf, (int)_len, _flags); }